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

2009-10-26 Thread James Chacon
On Oct 26, 2009, at 4:02 PM, David Laight wrote: On Sun, Oct 25, 2009 at 05:42:20PM +, 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 confusin

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

2009-10-26 Thread David Laight
On Sun, Oct 25, 2009 at 05:42:20PM +, 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.

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

2009-10-26 Thread Matthias Scheler
On Mon, Oct 26, 2009 at 06:14:36AM +, David Holland wrote: > On Sun, Oct 25, 2009 at 05:09:34PM +, David Laight wrote: > > Lint is differentially far too picky... > > Remove some warnings that only appear on i386 (not on amd64) and that > > for some reason best known to others are deemed