On 12.10.2007 15:05, Peter Stuge wrote:
> On Fri, Oct 12, 2007 at 02:54:41PM +0200, Carl-Daniel Hailfinger wrote:
>   
>> The only added code for all new functionality is
>>
>> +            if (walk < (char *)header) {
>> +                    walk = (char *)header;
>> +                    walk += (sizeof(struct lar_header) + strlen(fullname))
>> +                            & 0xfffffff0;
>> +            }
>>
>> and all other changes are either better code comments or
>> functionally equivalent (whitespace / argument reordering)
>> transformations.
>>     
>
> Please make several patches out of that. Easier to review then. :)
>   

Will do. Thanks for the review.

Regards,
Carl-Daniel

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



-- 
linuxbios mailing list
[email protected]
http://www.linuxbios.org/mailman/listinfo/linuxbios

Reply via email to