Source-Changes archive

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

Re: CVS commit: src/lib/libm/src



On Mon, Jan 19, 2009 at 09:19:23PM +0100, Matthias Drochner wrote:
> 
> lukem%NetBSD.org@localhost said:
> > Modified Files:
> >     src/lib/libm/src: e_jnf.c
> > Log Message: fix -Wsign-compare issue 
> 
> In the cases I looked at it would make more sense
> to change the variables in question to "unsigned int"
> than to add typecasts.

Yes, I dislike casts of intergal types.  The can hide more bugs
that compiler warnings.

I sometimes just add '+ 0u' to the signed integer.

        David

-- 
David Laight: david%l8s.co.uk@localhost


Home | Main Index | Thread Index | Old Index