DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=subject:from:reply-to:to:cc:in-reply-to:references:content-type
         :date:message-id:mime-version:x-mailer:content-transfer-encoding;
        b=IRHpKDlGIMctOPijdLPIJCO5sNT/WXzSS61vwABR/94HaixXeAHlYJ3ZKzDSRorcoy
         xA6q05Gpw651CxiM/CQq3VQBdotbLkHOV5VSZ3Orvs46kWUkfPWsadyl7HUC7xKPLzt1
         +HTLbEP0Zhp+1ODkTi2dN3QQsrV/rRGRv2CCM=
Subject: Re: [PATCH] fs/jffs2/xattr: Fix coding style
From: Artem Bityutskiy <dedekind1@gmail.com>
Reply-To: dedekind1@gmail.com
To: Tracey Dent <tdent48227@gmail.com>
Cc: dwmw2@infradead.org, linux-mtd@lists.infradead.org,
        linux-kernel@vger.kernel.org
In-Reply-To: <20110221093902.GA4759@linux-rdzc.hsd1.mi.comcast.net>
References: <20110221093902.GA4759@linux-rdzc.hsd1.mi.comcast.net>
Content-Type: text/plain; charset="UTF-8"
Date: Fri, 25 Feb 2011 12:51:14 +0200
Message-ID: <1298631074.2798.51.camel@localhost>
Mime-Version: 1.0
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 919
Lines: 27

On Mon, 2011-02-21 at 04:39 -0500, Tracey Dent wrote:
> Remove the trailing white space and get rid of the
> unnessary {} on certain statements.
> 
> Signed-off-by: Tracey Dent <tdent48227@gmail.com>

Could you please do one change per patch?

Make first patch to only fix the trailing white-space issue. I'd take it
to my tree immediately. You can make the second patch which removes
unnecessary {}, but this patch would be arguable, I'm not sure I'd take
it.

In any case, now I should reject whole patch, but in the "one patch per
logical change" case I'd take at least part of your work.

Thanks a lot!

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)

--
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/