Source-Changes-D archive

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

Re: CVS commit: src/lib/libc



On Tue, Mar 13, 2012 at 05:13:51PM -0400, Christos Zoulas wrote:
> Log Message:
> PR/45989: Martin Husemann: lint invocation does include -w only on i386
> 
> - turn lint -w for all the platforms after fixing the lint warnings.
> - add _DIAGASSERTS() for casts that would assign values to types that
>   would not fit.
> - change types, add casts
> - change into ansii prototypes
> - turn on _DIAGNOSTIC for libc (during current, to be eliminated for release
>   builds)

Thanks for ignoring my comments. It's especially useful that you
haven't really addressed any of the points. Checking casts from size_t
to uint32_t is useless on a good number of our architectures. Other
places have proper checks already in place, so this just adds bloat.

Joerg


Home | Main Index | Thread Index | Old Index