avcodec/mpegvideo: use predefined M_PI
authorGanesh Ajjanagadde <gajjanagadde@gmail.com>
Sat, 14 Nov 2015 15:49:45 +0000 (10:49 -0500)
committerGanesh Ajjanagadde <gajjanagadde@gmail.com>
Sat, 14 Nov 2015 15:58:39 +0000 (10:58 -0500)
M_PI is defined in math.h, or in avutil/mathematics.h for compatibility
hacks. This uses this value instead of an ad-hoc floating literal.

Reviewed-by: Michael Niedermayer <michael@niedermayer.cc>
Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com>
libavcodec/mpegvideo.c

index 96634ec..69e0595 100644 (file)
@@ -1870,8 +1870,8 @@ void ff_print_debug_info2(AVCodecContext *avctx, AVFrame *pict, uint8_t *mbskip_
                     uint64_t u,v;
                     int y;
 #define COLOR(theta, r) \
-    u = (int)(128 + r * cos(theta * 3.141592 / 180)); \
-    v = (int)(128 + r * sin(theta * 3.141592 / 180));
+    u = (int)(128 + r * cos(theta * M_PI / 180)); \
+    v = (int)(128 + r * sin(theta * M_PI / 180));
 
 
                     u = v = 128;