[vlc-commits] IVTC: fix undefined arithmetic

Rémi Denis-Courmont git at videolan.org
Thu May 5 22:16:03 CEST 2011


vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Thu May  5 23:15:34 2011 +0300| [2b58c0ef5331b2f159723b9c46f6696731c8d69b] | committer: Rémi Denis-Courmont

IVTC: fix undefined arithmetic

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=2b58c0ef5331b2f159723b9c46f6696731c8d69b
---

 modules/video_filter/deinterlace/algo_ivtc.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/modules/video_filter/deinterlace/algo_ivtc.c b/modules/video_filter/deinterlace/algo_ivtc.c
index 1498685..cf08e20 100644
--- a/modules/video_filter/deinterlace/algo_ivtc.c
+++ b/modules/video_filter/deinterlace/algo_ivtc.c
@@ -1251,7 +1251,7 @@ static bool IVTCOutputOrDropFrame( filter_t *p_filter, picture_t *p_dst )
             if( op == IVTC_OP_DROP_FRAME )
             {
                 /* Bump cadence counter into the next expected position */
-                p_ivtc->i_cadence_pos = ++p_ivtc->i_cadence_pos % 5;
+                p_ivtc->i_cadence_pos = (p_ivtc->i_cadence_pos + 1) % 5;
 
                 /* Drop frame. We're done. */
                 return false;
@@ -1385,7 +1385,7 @@ static bool IVTCOutputOrDropFrame( filter_t *p_filter, picture_t *p_dst )
         }
 
         /* Bump cadence counter into the next expected position. */
-        p_ivtc->i_cadence_pos = ++p_ivtc->i_cadence_pos % 5;
+        p_ivtc->i_cadence_pos = (p_ivtc->i_cadence_pos + 1) % 5;
     }
     else if( p_ivtc->i_mode == IVTC_MODE_TELECINED_NTSC_SOFT )
     {



More information about the vlc-commits mailing list