PATCH: minor cleanup to dwarf2read.c

Eli Zaretskii eliz@is.elta.co.il
Wed Jul 4 02:10:00 GMT 2001


On 4 Jul 2001, Jim Blandy wrote:

> If you have:
> 
>         foo_buf = malloc (sizeof (*foo_buf) * foo_length);
> 
>         for (i = 0; i < foo_length; i++)
>           ...
> 
> you don't put any scary comments around the for statement about the
> termination conditions, do you?

I meant something like this:

     /* The while loop below was originally this:

        while ((unsigned int) (info_ptr - dwarf_info_buffer)
              ((info_ptr - dwarf_info_buffer) % 4) < dwarf_info_size)

        This seems to be trying to round info_ptr up to the next 
        four-byte boundary, but that's not what it actually did.  If we 
        discover the problem the old code was really trying to address, 
        we can fix it properly.  */



More information about the Gdb-patches mailing list