Messages in this thread | | | Date | Mon, 23 Sep 2002 11:14:47 -0400 | From | Karim Yaghmour <> | Subject | Re: [PATCH] LTT for 2.5.37 2/9: Trace driver |
| |
Pavel Machek wrote: > Hi! > > > +/* Driver */ > > +static int sMajorNumber; /* Major number of the tracer */ > > +static int sOpenCount; /* Number of times device is open */ > > +/* Locking */ > > Why *s*OpenCount? Some creeping infection by hungarian notation?
"static" as opposed 'l' for "local" elsewhere in the code, but you're looking at code that is going to change to fit the usual kernel coding style. So all those S's and L's will go away ;)
Karim
=================================================== Karim Yaghmour karim@opersys.com Embedded and Real-Time Linux Expert =================================================== - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |