Hi, On Tue, Nov 24, 2009 at 5:04 AM, Shawn O. Pearce <spearce@xxxxxxxxxxx> wrote: > If I understand the change right, the only thing that really matters > here is removing the extra ';' in the if (max < avail) condition. > > That bug was the only reason why rpc->len < rpc->pos, and is thus > the only reason why avail would "go negative" (which it can't do > since its unsigned, it just wrapped around to a massive value). > [snip] > Right. This bug is really bad and should be fixed. But the message > above make this some little after-thought and doesn't explain what > was wrong with this being here. point noted. -- Cheers, Ray Chuan -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html