Karen Tung wrote:
> Hi,
>
> Please review my changes for bug 2263: microroot overflows with latest 
> changes to IPS
>
> http://defect.opensolaris.org/bz/show_bug.cgi?id=2263
>
> webrev is at:
>
> http://cr.opensolaris.org/~ktung/2263_fix/
>
> Thanks,
>
> --Karen
>
> _______________________________________________
> caiman-discuss mailing list
> caiman-discuss at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
>   
Karen:

57:  It is better to search for the whole string 
'flush-content-cache-on-success' since it may not work if a new name 
starts with 'flush-content-cache' is added to the cfg_cache file in the 
future.
58: not aligned properly
63: The status value may be 0 since it was set in line 36

- Sundar

Reply via email to