Re: [PATCH 00/24] drivers/block/floppy.c cleanups

[Stephen Hemminger]

On Fri, 22 Jan 2010 11:46:53 +0000
Alan Cox <alan@xxxxxxxxxxxxxxxxxxx> wrote:

> > Philosophically I'm not sure it's that great an idea to make bad code
> > look better without actually fixing/rewriting it. It's like putting
> > lipstick on a pig.
> 
> Unpacking the mess and sorting out the whacky macros is more like
> removing all muck from the mechanism so you can see what needs to be done.
> 
> So I disagree - its a useful first step only

I had a reason in motivating this. For some cases, customers use floppy as
a configuration storage, and there were a number of strange errors
showing up occasionally in recent kernels. But the driver is such
an old crufty mess, it wasn't worth investigating further.

Making the code clearer gets rid of that excuse.

-- 
--
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/