[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