Small cleanup inside setup_loopfilter function.

Change-Id: If7fa8aea02f26c2c2bb5daf4e65c3e661d7031ca

diff --git a/vp9/common/vp9_loopfilter.h b/vp9/common/vp9_loopfilter.h
index 458afc50bc76676b2d00dbf265c97eff81b3813f..bd9a35125b71f3be9945508c024e971444cd4922 100644 (file)
--- a/vp9/common/vp9_loopfilter.h
+++ b/vp9/common/vp9_loopfilter.h
@@ -20,7 +20,7 @@
 typedef enum {
   NORMAL_LOOPFILTER = 0,
   SIMPLE_LOOPFILTER = 1
-} LOOPFILTERTYPE;
+} LOOPFILTER_TYPE;
 
 #define SIMD_WIDTH 16
 

diff --git a/vp9/common/vp9_onyxc_int.h b/vp9/common/vp9_onyxc_int.h
index 26c0b8f83d6ca898229871c1023d7d63059ec768..a6a9806671ca8d4ade05d43f2e6041510175fd8b 100644 (file)
--- a/vp9/common/vp9_onyxc_int.h
+++ b/vp9/common/vp9_onyxc_int.h
@@ -240,7 +240,7 @@ typedef struct VP9Common {
   unsigned char *last_frame_seg_map;
 
   INTERPOLATIONFILTERTYPE mcomp_filter_type;
-  LOOPFILTERTYPE filter_type;
+  LOOPFILTER_TYPE filter_type;
 
   loop_filter_info_n lf_info;
 

diff --git a/vp9/decoder/vp9_decodframe.c b/vp9/decoder/vp9_decodframe.c
index f6f87efaef0c451a1e6d708c56b3f69acf46867f..a2bd3413b8e95d544243b57a8222d7223e39328f 100644 (file)
--- a/vp9/decoder/vp9_decodframe.c
+++ b/vp9/decoder/vp9_decodframe.c
@@ -1363,7 +1363,7 @@ static void setup_pred_probs(VP9_COMMON *pc, vp9_reader *r) {
 }
 
 static void setup_loopfilter(VP9_COMMON *pc, MACROBLOCKD *xd, vp9_reader *r) {
-  pc->filter_type = (LOOPFILTERTYPE) vp9_read_bit(r);
+  pc->filter_type = (LOOPFILTER_TYPE) vp9_read_bit(r);
   pc->filter_level = vp9_read_literal(r, 6);
   pc->sharpness_level = vp9_read_literal(r, 3);
 
@@ -1380,31 +1380,25 @@ static void setup_loopfilter(VP9_COMMON *pc, MACROBLOCKD *xd, vp9_reader *r) {
   xd->mode_ref_lf_delta_enabled = vp9_read_bit(r);
 
   if (xd->mode_ref_lf_delta_enabled) {
-    // Do the deltas need to be updated
     xd->mode_ref_lf_delta_update = vp9_read_bit(r);
-
     if (xd->mode_ref_lf_delta_update) {
       int i;
 
-      // Send update
       for (i = 0; i < MAX_REF_LF_DELTAS; i++) {
         if (vp9_read_bit(r)) {
-          // sign = vp9_read_bit(r);
-          xd->ref_lf_deltas[i] = vp9_read_literal(r, 6);
-
+          int value = vp9_read_literal(r, 6);
           if (vp9_read_bit(r))
-            xd->ref_lf_deltas[i] = -xd->ref_lf_deltas[i];  // Apply sign
+            value = -value;
+          xd->ref_lf_deltas[i] = value;
         }
       }
 
-      // Send update
       for (i = 0; i < MAX_MODE_LF_DELTAS; i++) {
         if (vp9_read_bit(r)) {
-          // sign = vp9_read_bit(r);
-          xd->mode_lf_deltas[i] = vp9_read_literal(r, 6);
-
+          int value = vp9_read_literal(r, 6);
           if (vp9_read_bit(r))
-            xd->mode_lf_deltas[i] = -xd->mode_lf_deltas[i];  // Apply sign
+            value = -value;
+          xd->mode_lf_deltas[i] = value;
         }
       }
     }