Re: [PATCH] fs/fcntl.c - remove impossible <0 check in do_fcntl -arg is unsigned.
From: Hans Reiser
Date: Thu Jan 08 2004 - 06:11:38 EST
Code is like poetry --- bloat needs to come out of it, and a rigorous
discipline of removing all unnecessary complexity from it needs to be
made a habit so that the exact meaning of it will be seen by all.
Boy was that pompous.;-) Oh well, I'd like the reiserfs portions of his
patch to be accepted, and I'll forward them to you in a few days.
Please consider accepting them.
Hans
Jesper Juhl wrote:
On Wed, 7 Jan 2004, Linus Torvalds wrote:
The fact that the compiler can optimize
away one of the tests if the type is right i2Ds fine. It seems to be
draconian to remove code that is correct and safe, especially when the
code has no real downsides to it.
--
Hans
-
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/