avcodec/libxvid: set lumi_aq for lumimasking
authorMichael Niedermayer <michaelni@gmx.at>
Mon, 5 Aug 2013 13:00:14 +0000 (15:00 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Mon, 5 Aug 2013 13:06:04 +0000 (15:06 +0200)
Simplifies check and should fix lumi+vari warning

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
libavcodec/libxvid.c

index edf5f43da8d1413fffddad54a73340dbf6a6133c..d1d411424914416289db3cf310c8b82332c9810f 100644 (file)
@@ -531,14 +531,17 @@ static av_cold int xvid_encode_init(AVCodecContext *avctx)  {
         xvid_enc_create.num_plugins++;
     }
 
         xvid_enc_create.num_plugins++;
     }
 
+    if ( avctx->lumi_masking != 0.0)
+        x->lumi_aq = 1;
+
     /* Luminance Masking */
     /* Luminance Masking */
-    if( avctx->lumi_masking != 0.0 || x->lumi_aq ) {
+    if( x->lumi_aq ) {
         masking_l.method = 0;
         plugins[xvid_enc_create.num_plugins].func = xvid_plugin_lumimasking;
 
         /* The old behavior is that when avctx->lumi_masking is specified,
          * plugins[...].param = NULL. Trying to keep the old behavior here. */
         masking_l.method = 0;
         plugins[xvid_enc_create.num_plugins].func = xvid_plugin_lumimasking;
 
         /* The old behavior is that when avctx->lumi_masking is specified,
          * plugins[...].param = NULL. Trying to keep the old behavior here. */
-        plugins[xvid_enc_create.num_plugins].param = x->lumi_aq ? &masking_l : NULL ;
+        plugins[xvid_enc_create.num_plugins].param = avctx->lumi_masking ? NULL : &masking_l ;
         xvid_enc_create.num_plugins++;
     }
 
         xvid_enc_create.num_plugins++;
     }