From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?iso-8859-1?Q?J=F6rn?= Engel Subject: Re: [PATCH 1/5] Forking ext4 filesystem from ext3 filesystem Date: Thu, 10 Aug 2006 18:48:32 +0200 Message-ID: <20060810164832.GA305@wohnheim.fh-wedel.de> References: <1155172622.3161.73.camel@localhost.localdomain> <20060809233914.35ab8792.akpm@osdl.org> <44DB61D7.1000109@us.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Andrew Morton , linux-kernel@vger.kernel.org, ext2-devel@lists.sourceforge.net, linux-fsdevel@vger.kernel.org Return-path: Received: from wohnheim.fh-wedel.de ([213.39.233.138]:43151 "EHLO wohnheim.fh-wedel.de") by vger.kernel.org with ESMTP id S1161455AbWHJQsr (ORCPT ); Thu, 10 Aug 2006 12:48:47 -0400 To: Mingming Cao Content-Disposition: inline In-Reply-To: <44DB61D7.1000109@us.ibm.com> Sender: linux-fsdevel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Thu, 10 August 2006 09:41:59 -0700, Mingming Cao wrote: >=20 > I agree with you that the timing is right, to do the clean up now rat= her=20 > than later. I would give it a try. If I could get more help from more= =20 > code reviewer, it probably makes the effort a lot easier. For those=20 > issues you pointed out : coding style problem??incorrect comments,=20 > poorly-named variables -- do you have any specific examples in your = mind? =46or whitespace damage, you can try the following regex: /\s\+$\| \+\ze\t/ Or if you use vim as an editor, you can add this to your vimrc: highlight RedundantSpaces ctermbg=3Dred guibg=3Dred match RedundantSpaces /\s\+$\| \+\ze\t/ =46or example, it will show 4 cases of trailing whitespace in the quote= d part of your email above. :) J=F6rn --=20 You ain't got no problem, Jules. I'm on the motherfucker. Go back in there, chill them niggers out and wait for the Wolf, who should be coming directly. -- Marsellus Wallace - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel= " in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html