Thank you, Jan, for your comments.
On 04/29/09 01:07, Jan Damborsky wrote:
> 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.
>
I'll push after making this change.
Jack
> 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
>
