lgtm.
ed
On Mon, Dec 21, 2009 at 05:41:29PM -0800, Jordan Vaughan wrote:
> That's a good idea. I updated the webrev.
>
> Thanks,
> Jordan
>
> On 12/21/09 05:08 PM, Steve Lawrence wrote:
> >Minor nit. You could use != POC_STRING, put the Z_NO_ENTRY in the {}, and
> >put the success case after. Not a required change.
> >
> >LGTM.
> >
> >-Steve
> >
> >On Fri, Dec 18, 2009 at 07:28:52PM -0800, Jordan Vaughan wrote:
> >>I expanded my webrev to include my fix for
> >>
> >>6910339 zonecfg coredumps with badly formed 'select net defrouter'
> >>
> >>I need someone to review my changes. The webrev is still accessible via
> >>
> >>http://cr.opensolaris.org/~flippedb/onnv-zone2
> >>
> >>Thanks,
> >>Jordan
> >>_______________________________________________
> >>zones-discuss mailing list
> >>zones-discuss@opensolaris.org
>
> _______________________________________________
> zones-discuss mailing list
> zones-discuss@opensolaris.org
_______________________________________________
zones-discuss mailing list
zones-discuss@opensolaris.org
