Re: [coreboot] lar directory handling patch

[Peter Stuge]

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

Like it! Just some small nits about coding style.

Acked-by: Peter Stuge
> 		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