Re: [PATCH] Fix NR_KEYS off-by-one error

From: Paul Jackson
Date: Thu Jul 29 2004 - 04:52:00 EST


> I didn't see any big cleanup in your patch.

That could well be <grin>. Beauty is in the eye of the beholder.

I figured that replacing the #define's with local variables
was worth a couple of style points, and that explicitly
checking that the user provided values were within limits,
all the time, no ifdefs, was more straight forward.

But it is difficult to establish such preferences with certainty.

Again, thanks for considering my reply.

Do as you think best. I am afraid I have no more wisdom (if
ever I had any ...).

--
I won't rest till it's the best ...
Programmer, Linux Scalability
Paul Jackson <pj@xxxxxxx> 1.650.933.1373
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/