Lizhong Li wrote:
> 
> I just updated the webrev according your comments, could you kindly help
> me review it ?
> 
> http://cr.opensolaris.org/~uniopen/mrtg/

Mainly looks okay to me, but ...

as Amanda pointed out the man Interface Stability level in 
sunman-stability and rateup.1 (& mrtg-traffic-sum.1) are different??

also should rateup.1 & mrtg-traffic-sum.1 have the CDDL stuff in them?

paul
-- 
----------------------------------------------------------------------
Paul Cunningham
Software Engineer
Tadpole Business Unit

Reply via email to