Re: [PATCH] Run cpuid.c and msr.c through Lindent

From: Brian Gerst
Date: Mon Feb 23 2004 - 19:43:44 EST


Chris Wedgwood wrote:
On Sat, Feb 21, 2004 at 07:16:32PM -0500, Brian Gerst wrote:


Run cpuid.c and msr.c through Lindent to improve readability. The
only non-whitespace change was to add a missing semicolon after
module_exit().


As much as I had poor/inconsistent formatting, gratuitous whitespace
changes are really annoying when you have to deal with merging code
across several kernel versions (as I have had to do) and I really
prefer to see these things done as the code is fixed/modified.

It's excuses like this that have allowed this crap to endure for so long without getting fixed. What patch do you have for these files that is conflicting? I haven't seen any proposed patches for these two files in a long time, and BK only shows one minor change in the last 6 months, so I didn't feel that I was going to step on anyone's toes with this patch.

--
Brian Gerst
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/