[FFmpeg-devel] [PATCH 5/5] lavc/snowenc: replace rint by lrint

Ganesh Ajjanagadde gajjanag at mit.edu
Sun Dec 27 05:31:32 CET 2015


On Fri, Dec 25, 2015 at 5:48 PM, Ganesh Ajjanagadde <gajjanag at mit.edu> wrote:
> On Fri, Dec 25, 2015 at 5:38 PM, Michael Niedermayer
> <michael at niedermayer.cc> wrote:
>> On Fri, Dec 25, 2015 at 04:00:20PM -0800, Ganesh Ajjanagadde wrote:
>>> avoids float to int cast, and is
>>
>>
>>> slightly superior in terms of rounding
>>> ("Dutch/Gauss rounding").
>>
>> isnt the rounding the same ?
>
> oops, confused with round. There is some difference regarding
> exceptions, rint throws an inexact exception when return differs from
> arg.
>
>>
>> the change itself LGTM

patch series pushed, thanks

>>
>> [...]
>> --
>> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>>
>> No great genius has ever existed without some touch of madness. -- Aristotle
>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>


More information about the ffmpeg-devel mailing list