HI Ethan.

pep8 requests a space after the ":".  Otherwise, LGTM.

    Thanks,
    Jack

On 09/06/11 16:10, Ethan Quach wrote:
Can I get a quick review for:


Webrev:
----------
https://cr.opensolaris.org/action/browse/caiman/equach/7087592/webrev/

BugID:
--------
7087592 <http://monaco.us.oracle.com/detail.jsf?cr=7087592> slim_install_unit test failure after 7086390 <http://monaco.us.oracle.com/detail.jsf?cr=7086390> is integrated.


thanks,
-ethan


_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to