On (09/22/09 08:48), James Carlson wrote: > > > > http://cr.opensolaris.org/~sowmini/routed_fixes/ > > > > which now handles both rip and rdisc, for all prefixes (including > > default) > > Nice! > > Bug at line 1262: that should be "%s", not "%x". > > Nit at line 1256 -- the message should say something about why it's > reporting a lack of a route. (got RTM_LOSING, but ...) Similar issue > with 1270. Nits at 1263 and 1270: \n and trailing spaces probably not > needed. >
Thanks! all the above nits are fixed. Webrev updated. --sowmini _______________________________________________ networking-discuss mailing list [email protected]
