lkml.org 
[lkml]   [2006]   [Oct]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 5/5] fdtable: Extensive fs/file.c cleanups.
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/

\
 
 \ /
  Last update: 2006-10-06 22:25    [W:0.274 / U:0.036 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site