On Wed, 1 November 2006 13:02:12 -0500, Holden Karau wrote: > On 11/1/06, Jörn Engel <joern@xxxxxxxxxxxxxxxxxxxx> wrote: > > > >Result would be something like: > > c_bh = kmalloc(... > > err = -ENOMEM; > > if (!c_bh) > > goto error; > That wouldn't work so well since we always return err, I don't quite follow. If the branch is taken, err is -ENOMEM. If the branch is not taken, err is set to 0 with the next instruction. Both methods definitely work. Whether one is preferrable over the other is imo 90% taste and maybe 10% better code on some architecture. So just pick what you prefer. Jörn -- Unless something dramatically changes, by 2015 we'll be largely wondering what all the fuss surrounding Linux was really about. -- Rob Enderle - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html