[coreboot] r3762 - trunk/coreboot-v2/src/mainboard/tyan/s2895
Myles Watson
mylesgw at gmail.com
Thu Nov 20 20:51:24 CET 2008
> -----Original Message-----
> From: coreboot-bounces at coreboot.org [mailto:coreboot-bounces at coreboot.org]
> On Behalf Of svn at coreboot.org
> Sent: Thursday, November 20, 2008 12:26 PM
> To: coreboot at coreboot.org
> Subject: [coreboot] r3762 - trunk/coreboot-v2/src/mainboard/tyan/s2895
>
> Author: stepan
> Date: 2008-11-20 20:26:16 +0100 (Thu, 20 Nov 2008)
> New Revision: 3762
>
> Modified:
> trunk/coreboot-v2/src/mainboard/tyan/s2895/cache_as_ram_auto.c
> Log:
> OK, people, watch this.
>
> This is a school book example of why trivial indent patches just suck
> big time.
>
> This error was introduced by a trivial self-acked indent patch and was
> never detected (because of a missing Config-abuild.lb)
>
> So, indenting the code for no reason can make it a lot worse (read:
> break it) instead of improving it.
>
> I ask everyone to keep this in mind when going on indent-frenzy again.
Guilty. I apologize.
Thanks,
Myles
More information about the coreboot
mailing list