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.

The changes I made fixed the -Wsign-compare warnings,
and resulted in no difference in the disassebled .o file.

I was leery of making more detailed changes to these
libm files because I couldn't find a decent test suite
in tree to verify the code and I am concerned about
breaking these.

If you want to improve this stuff, go ahead :)

cheers,
Luke.

Attachment: pgpheOmRs_hFh.pgp
Description: PGP signature



Home | Main Index | Thread Index | Old Index