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