This looks okay to me (assuming you handle Jim's comments)

Jim's last comment about the man pages - made me think; what are all the 
extra 'man' page files for that don't seem to match to a binary, etc., - 
are they all required (just checking)

Paul

Lizhong Li wrote:
> 
> I just modified the code according your comments.
> And the new webrev is available, please kindly help check,
> http://cr.opensolaris.org/~uniopen/mrtg
> 

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

Reply via email to