Hi Jesse
Thanks for the review. I have fixed the nits you provided.
Please find the updated webrev :
webrev :
https://cr.opensolaris.org/action/browse/caiman/nirmal27/6996539-2/webrev/original/
Diff :
https://cr.opensolaris.org/action/browse/caiman/nirmal27/6996539-2-diff/webrev/differential/
On 04/06/12 00:36, Jesse Butler wrote:
line 509:
nit - There are likely cleaner ways to do this. Given we have about 8
different places in the code where we get a netmask, it might be a good idea to
see if we can put this in one place and call it. Maybe a low-priority CR for
this would be good?
If you are fine with it, I would like to do this as a part of different CR.
Regards
Nirmal
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
