Messages in this thread | | | Subject | Re: [PATCH] fs/jffs2/xattr: Fix coding style | From | Artem Bityutskiy <> | Date | Fri, 25 Feb 2011 12:51:14 +0200 |
| |
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/
| |