On Wed, 2015-03-11 at 15:01 -0400, Nicholas Mc Guire wrote: > Putting code into the parameter list of wait_event_timeout() might be > legal C-code but not really readable - the "inline" code is simply > moved into a function and that passed to wait_event_timeout() as the > condition.
Arguably, that's even more unreadable since if you don't know this macro well you might assume the function is called only once, which is clearly not true... Don't get me wrong, I'm not opposed to this change, but if you ask me it's not completely clear that this makes it more readable. johannes -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
