[PATCH] Fix undefined behaviour in lrint,llrint.

Dave Korn dave.korn.cygwin@gmail.com
Sat Jul 31 00:36:00 GMT 2010


On 27/07/2010 00:59, Howland Craig D (Craig) wrote:
> As I had pointed out in the email thread concerning (possible) undefined
> shift behavior in lround and llround
> (http://sourceware.org/ml/newlib/2010/msg00437.html), lrint and llrint
> both have the same potential issue.  Attached is a patch--based on
> Dave's edits of lround--that fixes the problem.  The patch also corrects
> a minor typo in the comments for the previous lround and llround edits.

  Thanks for doing these functions (and saving me from having to), and for
catching my typo.  It all looked right to me but I didn't actually test it.

    cheers,
      DaveK




More information about the Newlib mailing list