[coreboot] r3762 - trunk/coreboot-v2/src/mainboard/tyan/s2895
Stefan Reinauer
stepan at coresystems.de
Thu Nov 20 21:22:24 CET 2008
Myles Watson wrote:
>
>> -----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.
;-) You have to break an egg to make an omelet.
I checked in a Config-abuild.lb that would expose such a breakage now.
All the best,
Stefan
--
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866
More information about the coreboot
mailing list