Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753508AbXEAQNM (ORCPT ); Tue, 1 May 2007 12:13:12 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753827AbXEAQNL (ORCPT ); Tue, 1 May 2007 12:13:11 -0400 Received: from mx1.redhat.com ([66.187.233.31]:53345 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753508AbXEAQNJ (ORCPT ); Tue, 1 May 2007 12:13:09 -0400 Organization: Red Hat UK Ltd. Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod Street, Windsor, Berkshire, SI4 1TE, United Kingdom. Registered in England and Wales under Company Registration No. 3798903 Directors: Michael Cunningham (USA), Charlie Peters (USA) and David Owens (Ireland) From: David Howells In-Reply-To: References: <20061127165138.GA2991@lst.de> <20070430040213.BF9901801A4@magilla.sf.frob.com> <20070430091121.GC31397@infradead.org> <20070430100917.439ebfc8.akpm@linux-foundation.org> <1178028973.2875.78.camel@pmac.infradead.org> To: "John Anthony Kazos Jr." Cc: David Woodhouse , Geert Uytterhoeven , Satyam Sharma , Andrew Morton , Christoph Hellwig , Roland McGrath , Christoph Hellwig , linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org Subject: Re: condingstyle, was Re: utrace comments X-Mailer: MH-E 8.0; nmh 1.1; GNU Emacs 22.0.50 Date: Tue, 01 May 2007 17:07:07 +0100 Message-ID: <9185.1178035627@redhat.com> Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 795 Lines: 28 John Anthony Kazos Jr. wrote: > if (veryverylengthycondition1 > && smallcond2 > && (conditionnumber3a > || condition3b)) { > ... > } > > Clear, crisp, and 80-wide. I also like how the logical operator on the > following line is indented slightly into the condition of the previous > line. The brain works better when associated things are lined up, so: if (veryverylengthycondition1 && smallcond2 && (conditionnumber3a || condition3b)) { is better as it gives a visual clue to association. David - 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/