Re: [PATCH] Avoid overflows in kernel/time.c (version 5)

From: Roman Zippel
Date: Sun Dec 23 2007 - 21:51:24 EST


Hi,

On Monday 17 December 2007, H. Peter Anvin wrote:

> kernel/timeconst.pl | 340

I agree with Jan that it would be better to put this into scripts.
In the long term we could also detect some of the simple special cases, so we
can finally inline some of these functions again, this would mean the
generated file would go into include/linux/ and the script should be in
scripts/.

> + my ($b,$n,$d) = @_;

I noticed this already in the old script, using something else than single
letter variables would increase readability considerably.

bye, Roman
--
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/