Ethan Quach wrote:
> Evan,
> 
> libbe.h
> ---------
> I don't see why do_print needs to be defined in here. libbe_priv.h seems 
> more appropriate.

You're correct it should have been defined in libbe_priv.h. This will need to 
be 
fixed.

> Otherwise looks okay.
> 
> -ethan
> 
> Evan Layton wrote:
>> I need a code review for:
>>
>> http://cr.opensolaris.org/~evanl/snap_996
>>
>> This webrev includes the fixes for 996 and 1000
>>
>> http://defect.opensolaris.org/bz/show_bug.cgi?id=996
>> http://defect.opensolaris.org/bz/show_bug.cgi?id=1000
>>
>> Thanks!
>> -evan
>> _______________________________________________
>> caiman-discuss mailing list
>> caiman-discuss at opensolaris.org
>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
>>   


Reply via email to