[FFmpeg-devel] [PATCH 01/10] avformat/mov: use hypot()

[Michael Niedermayer]

On Sun, Nov 22, 2015 at 12:05:41PM -0500, Ganesh Ajjanagadde wrote:
> This simplifies the code.
> 
> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
> ---
>  libavformat/mov.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/libavformat/mov.c b/libavformat/mov.c
> index 8493de9..5fc48cb 100644
> --- a/libavformat/mov.c
> +++ b/libavformat/mov.c
> @@ -3421,9 +3421,8 @@ static int mov_read_tkhd(MOVContext *c, AVIOContext *pb, MOVAtom atom)
>      if (width && height && sc->display_matrix) {
>          double disp_transform[2];
>  
> -#define SQR(a) ((a)*(double)(a))
>          for (i = 0; i < 2; i++)
> -            disp_transform[i] = sqrt(SQR(display_matrix[i][0]) + SQR(display_matrix[i][1]));
> +            disp_transform[i] = hypot(display_matrix[i][0], display_matrix[i][1]);

LGTM

thx

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

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 
-------------- 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/20151122/e47f2335/attachment.sig>