From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752730AbcFJIeT (ORCPT ); Fri, 10 Jun 2016 04:34:19 -0400 Received: from out1134-195.mail.aliyun.com ([42.120.134.195]:60387 "EHLO out1134-195.mail.aliyun.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751374AbcFJIeQ (ORCPT ); Fri, 10 Jun 2016 04:34:16 -0400 X-Greylist: delayed 316 seconds by postgrey-1.27 at vger.kernel.org; Fri, 10 Jun 2016 04:34:16 EDT X-Alimail-AntiSpam: AC=CONTINUE;BC=0.07882325|-1;FP=0|0|0|0|0|-1|-1|-1;HT=e02c03290;MF=chengang@emindsoft.com.cn;NM=1;PH=DS;RN=10;RT=10;SR=0;TI=SMTPD_----4ue74Ng_1465547329; Message-ID: <575A7BAF.60000@emindsoft.com.cn> Date: Fri, 10 Jun 2016 16:34:55 +0800 From: Chen Gang User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Michal Hocko CC: akpm@linux-foundation.org, trivial@kernel.org, vdavydov@virtuozzo.com, hannes@cmpxchg.org, davem@davemloft.net, tj@kernel.org, riel@redhat.com, linux-kernel@vger.kernel.org, Chen Gang Subject: Re: [PATCH trivial] include/linux/memcontrol.h: Clean up code only References: <1465485832-24175-1-git-send-email-chengang@emindsoft.com.cn> <20160609154605.GH24777@dhcp22.suse.cz> <575A0C7E.9020500@emindsoft.com.cn> <20160610061406.GA32285@dhcp22.suse.cz> In-Reply-To: <20160610061406.GA32285@dhcp22.suse.cz> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 6/10/16 14:14, Michal Hocko wrote: > On Fri 10-06-16 08:40:30, Chen Gang wrote: >> >> On 6/9/16 23:46, Michal Hocko wrote: > [...] >>> That's being said, I appreciate an interest in making the code cleaner >>> but try to think whether these changes are actually helpful and who is >>> going to benefit from them. >>> >> >> For me, another readers can get benefit more or less from it: if read a >> simple line can know the whole thing (function work), why need we read >> multiple lines? > > Yes I understand that this is a matter of taste but as I've said above. > I do not see this would add a benefit to justify the change. > > If you are doing a reformating or white space cleanups always try to > think about those who want/need to dig into the history to understand > the code and this would add an additional step to get to the original > commit which is added the code. This is just wasting of time. > > Now this would be much more tolerable when the code in question was a > maze but this is not the case. > OK, thanks. Cleaning up code in include/* should face to the whole header files, not only for mm specially. It is not suitable to only focus mm in common shared header files for cleaning up code only. So for me, cleaning up code in include/* is still necessary, but I shall face to all files instead of mm related files. Thanks. -- Chen Gang (陈刚) Managing Natural Environments is the Duty of Human Beings.