Sowmini.Varadhan at Sun.COM writes: > Meanwhile, here's the updated webrev, in case you (or anyone > else!) want to take another look: > > http://cr.opensolaris.org/~sowmini/nce_walker/
The new code looks pretty good. There are a couple of very minor nits that I'd pick (e.g., modules/ip/ip.c:1408,1419: why is the return inside the braces for one clause but not the other?), but I'm happy enough with it. -- James Carlson, Solaris Networking <james.d.carlson at sun.com> Sun Microsystems / 35 Network Drive 71.232W Vox +1 781 442 2084 MS UBUR02-212 / Burlington MA 01803-2757 42.496N Fax +1 781 442 1677
