From mboxrd@z Thu Jan  1 00:00:00 1970
From: Eli Zaretskii <eliz@is.elta.co.il>
To: Jim Blandy <jimb@zwingli.cygnus.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: PATCH: minor cleanup to dwarf2read.c
Date: Wed, 04 Jul 2001 02:10:00 -0000
Message-id: <Pine.SUN.3.91.1010704120651.4589B-100000@is>
References: <npy9q5nllr.fsf@zwingli.cygnus.com>
X-SW-Source: 2001-07/msg00041.html

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.  */