patch for binutils bfd/dwarf2.c

Nick Clifton nickc@redhat.com
Tue Sep 24 03:43:00 GMT 2002


Hi Nathan,

> While merging into my code the latest cvs changes for bfd/dwarf2.c, I
> noticed the following (re: the patch below).
> 
> At line 1136:
> 
>     arange_add (unit, low_pc, high_pc);       <-----
>     if (low_pc == 0 || address < low_pc)
>       low_pc = address;
>     if (address > high_pc)
>       high_pc = address;
>     arange_add (unit, low_pc, address);       <-----
> 
> 
> Here, the range is added twice, and with possibly overlapping ranges.  Since
> this was different than both the original code and my patch, and since it
> looked like a mistake, I wanted to make sure you verified it.

Oops - typo in my part.  I have checked in a fix.

Cheers
        Nick



More information about the Binutils mailing list