Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/lib/libc/stdio




On Oct 26, 2009, at 4:02 PM, David Laight wrote:

On Sun, Oct 25, 2009 at 05:42:20PM +0000, Christos Zoulas wrote:

Can we just revert the past 2 commits? Changing:

(size_t)x -> x + 0u

does not look like an improvement to me. At least the first shows the intent,
the second is just confusing, specially when size_t is unsigned long.

Actually, IMHO, lint is just being too picky here.
Having to add a cast whenever an 'int' variable is passed to a function
that has a 'size_t' parameter really is just polluting the code with
pointless casts.


They are different types though. It's a signed vs unsigned issue.

James



Home | Main Index | Thread Index | Old Index