[FFmpeg-devel] [PATCH 2/2] avutil/eval: change sqrt to hypot

Michael Niedermayer michael at niedermayer.cc
Sun Nov 15 03:19:56 CET 2015


On Sat, Nov 14, 2015 at 12:11:20PM -0500, Ganesh Ajjanagadde wrote:
> This improves the mathematical behavior of hypotenuse computation.
> 
> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
> ---
>  libavutil/eval.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

if a hypot() function fallback is added this is ok
otherwise the code as it is before this should be fine too in practice
doubles have a quite large exponent range, half of that is still quite
large

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151115/8152f2b5/attachment.sig>


More information about the ffmpeg-devel mailing list