Hi Jack, thank you very much for making those changes !
I have only nit related to Dave's comments - as Dave pointed out, the similar change might be done also on line 129, e.g. Warning: NWAM is not disabled. -> Warning: NWAM is enabled. Other than that the changes look good - no need for another round of code review. Jan Jack Schwartz wrote: > Thanks for your reviews, Jan and Dave. > > I actually thought about this in the shower last night, where I do my > best thinking... :) > > I have changed the message per your request. > > Was: > "Too many SMF network/physical services are not disabled." > Changed to: > "More than one SMF network/physical service is enabled." > > Webrev updated at same location: > http://cr.opensolaris.org/~schwartz/090427.1/webrev/ > > Delta webrev at: > http://cr.opensolaris.org/~schwartz/090427.1/webrev.incr_1_2 > > Thanks, > Jack
