On Tue, Jun 08, 2010 at 04:19:18PM -0400, Eric Paris wrote: > > + rc = cond_read_av_list(p, fp, &node->false_list, node->true_list); > > + if (rc < 0) > > goto err; > > I know that lots of the next_entry() calls use if (rc < 0) [unrelated > note I think those need to be fixed too] but most of the code uses if > (rc) and I strongly prefer if (rc) as it's cleaner and faster from a > micro architecture point of view. Would you mind re-spinning this > one? > > -Eric The thing I like about if (rc < 0) is that it's makes it explicit that we expect negative error values. My gut says this will make it easier for static checkers to track what the value of "rc" is. But I don't have strong feelings about this. I'll resend the whole set tomorrow or the next day with all the changes you suggested. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
