[coreboot] [PATCH] v2: Use ARRAY_SIZE everywhere

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Wed Sep 24 17:32:37 CEST 2008


On 26.08.2008 00:09, Uwe Hermann wrote:
> On Fri, Aug 22, 2008 at 04:18:09PM +0200, Carl-Daniel Hailfinger wrote:
>   
>> The ARRAY_SIZE macro is convenient, yet mostly unused. Switch lots of
>> code to use it. That makes the code more readable and also less error-prone.
>>
>> I'd appreciate test builds. I see v3 as the future and occassional
>> patches to v2 are only intended to keep v2 roughly up to date with v3
>> code changes.
>>
>> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>>     
>
> Looks good, but I only want to ack if this was build-tested with abuild
> and some sample payloads for all boards.
>   

Survived a full abuild run.

> Did you check that stdlib.h is included everywhere where ARRAY_SIZE is
> used after the patch? Otherwise I expect compile errors.
>   

Thanks for the hint, fixed in the tree. Do you want me to resend or is a
full abuild run sufficient?

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/





More information about the coreboot mailing list