[coreboot] lar directory handling patch

Peter Stuge peter at stuge.se
Thu Feb 28 01:11:58 CET 2008


On Wed, Feb 27, 2008 at 01:32:25PM -0700, Myles Watson wrote:
> Here's the updated patch.
> 
> Signed-off-by: Myles Watson <mylesgw at gmail.com>

Like it! Just some small nits about coding style.


Acked-by: Peter Stuge <peter at stuge.se>


> +				len += (name[len-1]=='/'?1:0);

Some like spaces around ? and :


> +	if (verbose()) printf("%s: %s:%s\n",__FUNCTION__,filename,pathname);

Separate lines and space after comma.


>  	// Is it a directory?

C++ comment.


> +		ret = handle_directory(filename,pathname,thisalgo);

Space after commas.


Personally I don't care about these things, terser is better for me,
but I guess we should try to keep the style consistent in the repo.


//Peter




More information about the coreboot mailing list