Messages in this thread | | | Date | Sun, 30 Sep 2012 16:32:25 +0900 | From | Tejun Heo <> | Subject | Re: [PATCH 04/12] workqueue: simplify is_chained_work() |
| |
Hello,
On Fri, Sep 28, 2012 at 05:52:02PM +0800, Lai Jiangshan wrote: > Main reason: I think the readability of your is the same as mine, > and your add two lines. > > Tiny reason: my code uses only one return. (I don't always keep one return, > but I try to keep it if it is still clean) > > Is there any other reason to change it?
I don't like that the same condition is tested twice but technical advantages on issues like this are usually too small to be used as the sole basis to choose one over another and it usually comes down to preferences. For little things without clear technical advantage, following maintainer's taste tends to lead to higher consistency. So, yeah, I'd appreciate if you change it on the next posting or will probably just change it while applying it.
Thanks.
-- tejun
| |