[ovs-dev] [PATCH] ovs-vsctl: Remove dead assignment.

Ethan Jackson ethan at nicira.com
Tue Mar 29 18:12:51 UTC 2011


Oooooh, I misunderstood your statement.  I thought you meant remove
that line of code entirely so the loop would be:

for (a;;c) {

I'm fine with moving it up.  I'll do that and merge.

On Tue, Mar 29, 2011 at 11:11 AM, Ben Pfaff <blp at nicira.com> wrote:
> On Tue, Mar 29, 2011 at 11:08:18AM -0700, Ethan Jackson wrote:
>> > Optionally you could move "row != NULL" up a line, too.
>>
>> Can I?  I can certainly change it to simply "row".  If I remove it
>> entirely won't it either loop forever or null dereference at some
>> point?
>
> The following are equivalent:
>    for (a; b;
>         c)
> and
>    for (a;
>         b;
>         c)
>
> That's all I was suggesting.
>



More information about the dev mailing list