Hi Jean.
I checked that you got all instances of "pfinstall", so that part is OK...
I'm wondering about the comment change on line 421 of
perform_slim_install.c. The function in which the comment appears is
not "pfinstall", so to me it doesn't seem correct to change "pfinstall"
to the name of the current function. Not sure if it might make more
sense just to remove the sentence altogether?
Thanks,
Jack
On 07/06/09 16:03, Jean McCormack wrote:
> I'd like a code review for 789 Remove the pfinstall references in
> liborchestrator
>
> Webrev:
> http://cr.opensolaris.org/~jeanm/slim_789/
>
> CR:
> http://defect.opensolaris.org/bz/show_bug.cgi?id=789
>
> Jean
> _______________________________________________
> caiman-discuss mailing list
> caiman-discuss at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
