lkml.org 
[lkml]   [2012]   [Sep]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [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


\
 
 \ /
  Last update: 2012-09-30 10:01    [W:0.093 / U:0.016 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site