Eddie Luo wrote:
>>> New webrev can be found at the same place: 
>>> http://cr.opensolaris.org/~eddie/lua/.
>>>
>>
>> Remove extra leading spaces in all CDDL headers
>>    2 #  CDDL HEADER START
>> ...
>> should be:
>>    2 # CDDL HEADER START
>> ...
> 
> Will fix. Enn, I read the header from usr/src/cmd/meld/Makefile.sfw so 
> it seems that it also has this problem.

I forgot. I'm going to update that soon. Maybe I will
use your webrev as an example :)

Looks good.

Good job!

Cheers,
Jim

-- 
Jim Walker, http://blogs.sun.com/jwalker
Sun Microsystems, Software, Solaris QE
x77744, 500 Eldorado Blvd, Broomfield CO 80021

Reply via email to