On Fri, 12 Nov 2010 08:24:57 +0800 Zhang Rui <rui.zhang@xxxxxxxxx> wrote: > On Thu, 2010-11-11 at 23:27 +0800, Alan Cox wrote: > > From: Alan Cox <alan@xxxxxxxxxxxxxxx> > > > > And while > > > touching that function definition > > agreed. > > > do something about the disaster of formatting there. > > > well. this is what we got after running Lindent. > Your changes are reverted If I apply your patch and run Lindent again. > So maybe this is a problem that should be fixed in Lindent? Lindent is just a script for indent which is designed to help a human not replace them. It's not something to run automatically over code as it makes bad mistakes now and then. Alan -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html