On Tue, 01 Dec 2009 18:07:37 -0800 Renee Danson Sommerfeld <renee.sommerfeld at sun.com> wrote:
> On Mon, Nov 30, 2009 at 02:02:52AM -0800, Michael Hunter wrote: > > /net/coupe.eng/builds/mph/nwam1-cr-fixes/webrev > > Note: I'm assuming webrev path is actually > > /net/coupe.eng/builds/mph/nwam1_cr_fixes/webrev/ > > (underscores instead of hyphens) > > > events.c > 489: s/schedule/scheduled/ ACCEPT > > 493: As none of the current set of callers require nanosecond granularity > (0 is passed for delta_ns in all cases), I think this parameter > should be removed and a value of 0 assumed. ACCEPT > > 528: Why is this here? Seems like this parameter should just be removed, > as this odd line is the only reference to it now. ACCEPT > > 548: This needs to be on two lines. ACCEPT > > -renee
