Jean McCormack wrote: > nit: lines 775 & 776 have wrong indentation. > Otherwise it looks fine to me. Updated webrev per Jean's comments: http://cr.opensolaris.org/~sjelinek/bug_8071/
Going to commit changes. thanks, sarah **** > > Jean > > Sarah Jelinek wrote: >> Could I please get a code review for the following: >> >> http://defect.opensolaris.org/bz/show_bug.cgi?id=8071 >> 8071 AI fails if 'username' and 'userpass' are both left out of SC >> manifest. >> >> Webrev: >> http://cr.opensolaris.org/~sjelinek/bug_8071/ >> >> thanks, >> sarah >> _______________________________________________ >> caiman-discuss mailing list >> caiman-discuss at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >
