Date: Fri, 6 Oct 2006 13:22:58 -0700
From: Andrew Morton <akpm@osdl.org>
To: Vadim Lobanov <vlobanov@speakeasy.net>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH 5/5] fdtable: Extensive fs/file.c cleanups.
Message-Id: <20061006132258.39fc58ed.akpm@osdl.org>
In-Reply-To: <200610052152.29013.vlobanov@speakeasy.net>
References: <200610052152.29013.vlobanov@speakeasy.net>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1052
Lines: 25

On Thu, 5 Oct 2006 21:52:28 -0700
Vadim Lobanov <vlobanov@speakeasy.net> wrote:

> As long as the previous patch replaces the guts of fs/file.c, it makes sense
> to tidy up all the code within. This work includes:
> 	code simplification via refactoring,
> 	elimination of unnecessary code paths,
> 	extensive commenting throughout the entire file, and
> 	other minor cleanups and consistency tweaks.
> This patch does not contain any functional modifications.
> 
> This is the last patch in the series. All the code should now be sparkly
> clean.

This (wordwrapped) patch should have been the first in the series, not the
last.  Having the substantive changes come after the cleanups makes
subsequent problem diagnosis simpler, means we have less code to revert if
it goes bad, etc.

So I'll drop this one.
-
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/