Turns out hypot fallback added recently is currently not needed in the sense that no regressions have been observed from earlier commit fa972dcf1a36805de4aaa031331061a0a334996d. However, I do not consider this sufficient justification for reverting associated configure checks and fallback, simply because there can very well exist platforms lacking this especially since it is C99, that are not tested by FATE.
Anyway, this series adds hypot whenever useful throughout FFmpeg by running a manual audit for sqrt usage and modifying accordingly. 8/10 was just a trivial thing I noticed while working on this stuff. Patches tested with FATE on x86-64, GNU/Linux. Ganesh Ajjanagadde (10): avformat/mov: use hypot() ffplay: use hypot() avcodec/aacps_tablegen: use hypot() avfilter/af_compand: use hypot() avfilter/avf_showcqt: use hypot() avfilter/vf_chromakey: use hypot() avfilter/vf_codecview: use hypot() avfilter/vf_colorkey: avoid unnecessary floating point multiply avfilter/vsrc_mandelbrot: use hypot() avfilter/vsrc_mptestsrc: use hypot() ffplay.c | 6 +++--- libavcodec/aacps_tablegen.h | 2 +- libavfilter/af_compand.c | 4 ++-- libavfilter/avf_showcqt.c | 2 +- libavfilter/vf_chromakey.c | 2 +- libavfilter/vf_codecview.c | 2 +- libavfilter/vf_colorkey.c | 2 +- libavfilter/vsrc_mandelbrot.c | 2 +- libavfilter/vsrc_mptestsrc.c | 2 +- libavformat/mov.c | 3 +-- 10 files changed, 13 insertions(+), 14 deletions(-) -- 2.6.2 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
