[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