On 25 February 2015 at 19:30, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > Gar. No, I don't care about "if (unlikely(err < 0)" or > "if (unlikely(err)) ". Those seem like petty things to me so I'm not > getting involved with the argument between the two of you. (My secret > real opinion, is that I doubt anyone benchmarked it so probably the > unlikely() annotations hurt readability for no good reason. In other > words, "if (err)" is correct.) Thanks for drawing attention to this code anyway. -- 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