Return-Path: <cyrus@holtmann.org>
Message-ID: <4FA78618.4050203@broadcom.com>
Date: Mon, 7 May 2012 10:21:44 +0200
From: "Arend van Spriel" <arend@broadcom.com>
MIME-Version: 1.0
To: "Andrei Emeltchenko" <andrei.emeltchenko.news@gmail.com>,
	"David Miller" <davem@davemloft.net>, dh.herrmann@googlemail.com,
	gustavo@padovan.org, linville@tuxdriver.com,
	linux-bluetooth@vger.kernel.org, linux-wireless@vger.kernel.org
Subject: Re: [PATCH] Bluetooth: Fix coding style
References: <1336157988-4602-1-git-send-email-gustavo@padovan.org>
 <20120506.123656.1335354317428258137.davem@davemloft.net>
 <CANq1E4TTfDxZPsOqqWUT7PqTKUr8nCTC-wo=iGe_pxoUmnecgQ@mail.gmail.com>
 <20120506.145304.1998242927519675381.davem@davemloft.net>
 <20120507081411.GA19002@aemeltch-MOBL1>
In-Reply-To: <20120507081411.GA19002@aemeltch-MOBL1>
Content-Type: text/plain;
 charset=iso-8859-1
Sender: linux-bluetooth-owner@vger.kernel.org
List-ID: <linux-bluetooth.vger.kernel.org>

On 05/07/2012 10:14 AM, Andrei Emeltchenko wrote:
>> They don't.  All the examples you'll find are of the form:
>> > 
>> > 	if (a &&
>> > 	    b)
>> > 
>> > not:
>> > 
>> > 	if (a &&
>> > 			b)
> Actually it does not look like this, otherwise "b" would be placed in the
> same line, don't it?

Let me rephrase Dave's remark so you may understand:

	if (a_has_to_be_long_enough_to_make_you_understand &&
	    you_do_not_care_about_code_readability)

and not:
	if (a_has_to_be_long_enough_to_make_you_understand &&
			you_do_not_care_about_code_readability)

Hope it helps.

Gr. AvS