On Sat, Dec 19, 2015 at 8:24 AM, Michael Niedermayer <mich...@niedermayer.cc> wrote: > On Sat, Dec 19, 2015 at 08:01:37AM -0800, Ganesh Ajjanagadde wrote: >> On Sat, Dec 19, 2015 at 2:45 AM, Michael Niedermayer >> <mich...@niedermayer.cc> wrote: >> > On Fri, Dec 18, 2015 at 09:45:27PM -0800, Ganesh Ajjanagadde wrote: >> >> lrint is faster and conveys the intent better here. It is safe as long >> >> int has >> >> at least 32 bits. >> >> >> >> Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com> >> >> --- >> >> libavfilter/vf_idet.c | 2 +- >> >> 1 file changed, 1 insertion(+), 1 deletion(-) >> > >> > half_life should be checked to be within the defined range of lrint() >> >> sorry, don't understand: exp2 of a negative number is always < 1, so >> the lrint operates on a number < PRECISION, which falls within lrint's >> range. > > right, sorry i misread the code > > the patch should be fine
pushed, thanks > > [...] > > -- > Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB > > Everything should be made as simple as possible, but not simpler. > -- Albert Einstein > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
