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

2009-10-26 Thread David Holland
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 fatal for i386. Making this code 'pass lint' does absolutely nothing

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 fatal

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 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