Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753273AbaDWF0n (ORCPT ); Wed, 23 Apr 2014 01:26:43 -0400 Received: from g4t3426.houston.hp.com ([15.201.208.54]:7897 "EHLO g4t3426.houston.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752959AbaDWFZu (ORCPT ); Wed, 23 Apr 2014 01:25:50 -0400 Message-ID: <1398230745.27667.2.camel@buesod1.americas.hpqcorp.net> Subject: Re: [PATCH 5/4] ipc,shm: minor cleanups From: Davidlohr Bueso To: "Michael Kerrisk (man-pages)" Cc: Manfred Spraul , Davidlohr Bueso , Martin Schwidefsky , LKML , Andrew Morton , KAMEZAWA Hiroyuki , KOSAKI Motohiro , gthelen@google.com, aswin@hp.com, linux-mm@kvack.org Date: Tue, 22 Apr 2014 22:25:45 -0700 In-Reply-To: <53574AA5.1060205@gmail.com> References: <1398090397-2397-1-git-send-email-manfred@colorfullife.com> <1398221636.6345.9.camel@buesod1.americas.hpqcorp.net> <53574AA5.1060205@gmail.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.6.4 (3.6.4-3.fc18) Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2014-04-23 at 07:07 +0200, Michael Kerrisk (man-pages) wrote: > On 04/23/2014 04:53 AM, Davidlohr Bueso wrote: > > - Breakup long function names/args. > > - Cleaup variable declaration. > > - s/current->mm/mm > > > > Signed-off-by: Davidlohr Bueso > > --- > > ipc/shm.c | 40 +++++++++++++++++----------------------- > > 1 file changed, 17 insertions(+), 23 deletions(-) > > > > diff --git a/ipc/shm.c b/ipc/shm.c > > index f000696..584d02e 100644 > > --- a/ipc/shm.c > > +++ b/ipc/shm.c > > @@ -480,15 +480,13 @@ static const struct vm_operations_struct shm_vm_ops = { > > static int newseg(struct ipc_namespace *ns, struct ipc_params *params) > > { > > key_t key = params->key; > > - int shmflg = params->flg; > > + int id, error, shmflg = params->flg; > > It's largely a matter of taste (and I may be in a minority), and I know > there's certainly precedent in the kernel code, but I don't much like the > style of mixing variable declarations that have initializers, with other > unrelated declarations (e.g., variables without initializers). What is > the gain? One less line of text? That's (IMO) more than offset by the > small loss of readability. Yes, it's taste. And yes, your in the minority, at least in many core kernel components and ipc. Thanks, Davidlohr -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/