On Sat, Jul 29, 2006 at 05:23:05PM -0500, Eric Sandeen wrote:
> Andi Kleen wrote:
> > Eric Sandeen <sandeen@xxxxxxxxxxx> writes:
> >
> >> This gets rid of some pointless macro defines... I had a version that
> >> lower-cased it all too but Nathan liked this better, and he's the man!
> >> :)
> >
> > Shouted function names is not exactly Linux code style at least.
> >
> > -Andi
> >
>
> well, *shrug* I have both versions, Nathan can take his pick :)
>
> honestly, one-liner static inlines isn't exactly linux code style either, tho
> the typechecking is nice.
>
> I guess I shouldn't have said "Nathan liked this better" - I think he was
> being
> pragmatic about the scope of the change.
Right, its more that we don't have a great track record at the moment
of not introducing regressions with these cleanups (including myself),
so I'm becoming more reluctant to do sweeping changes across the whole
codebase. Smaller, specific, and obviously-correct things are less
likely to introduce issues, so if we can achieve basically the same
thing while churning the code less, I'm all for it.
cheers.
--
Nathan
|