[FFmpeg-cvslog] postproc: Removed unecessary if/else branch when getting QP.

Tucker DiNapoli git at videolan.org
Sat Mar 28 02:01:43 CET 2015


ffmpeg | branch: master | Tucker DiNapoli <T.DiNapoli42 at gmail.com> | Fri Mar 27 16:51:43 2015 -0400| [303c3dada243243d4896c81979c15ce67b028cd6] | committer: Michael Niedermayer

postproc: Removed unecessary if/else branch when getting QP.

There's still an if, as QP needs to be modified if isColor=0, but it
still removes a unecessary branch.

Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=303c3dada243243d4896c81979c15ce67b028cd6
---

 libpostproc/postprocess_template.c |   12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/libpostproc/postprocess_template.c b/libpostproc/postprocess_template.c
index 4016b08..16e441a 100644
--- a/libpostproc/postprocess_template.c
+++ b/libpostproc/postprocess_template.c
@@ -3529,16 +3529,12 @@ static void RENAME(postProcess)(const uint8_t src[], int srcStride, uint8_t dst[
 
           for(x = startx; x < endx; x+=BLOCK_SIZE){
             const int stride= dstStride;
-
-            if(isColor){
-                QP= QPptr[x>>qpHShift];
-                c.nonBQP= nonBQPptr[x>>qpHShift];
-            }else{
-                QP= QPptr[x>>4];
+            QP = QPptr[x>>qpHShift];
+            c.nonBQP = nonBQPptr[x>>qpHShift];
+            if(!isColor){
                 QP= (QP* QPCorrecture + 256*128)>>16;
-                c.nonBQP= nonBQPptr[x>>4];
                 c.nonBQP= (c.nonBQP* QPCorrecture + 256*128)>>16;
-                yHistogram[ srcBlock[srcStride*12 + 4] ]++;
+                yHistogram[srcBlock[srcStride*12 + 4]]++;
             }
             c.QP= QP;
 #if TEMPLATE_PP_MMX



More information about the ffmpeg-cvslog mailing list