]> granicus.if.org Git - libvpx/commitdiff
warnings in various experiments
authorJim Bankoski <jimbankoski@google.com>
Fri, 30 Nov 2012 15:29:43 +0000 (07:29 -0800)
committerGerrit Code Review <gerrit@gerrit.golo.chromium.org>
Fri, 30 Nov 2012 15:31:37 +0000 (07:31 -0800)
Change-Id: Ib5106d4772450f8026f823dd743f162ab833b1d6

vp9/common/vp9_reconintra.c
vp9/common/vp9_reconintra4x4.c
vp9/common/vp9_rtcd_defs.sh
vp9/decoder/vp9_decodframe.c
vp9/decoder/vp9_idct_blk.c
vp9/encoder/vp9_bitstream.c
vp9/encoder/vp9_encodeframe.c
vp9/encoder/vp9_onyx_int.h
vp9/encoder/vp9_rdopt.c

index d2f7e701af02daf4aab85ba42104a525b300fffb..5c8d339603a24fbbbe5bff2289ee11f0cbc1435a 100644 (file)
@@ -624,9 +624,7 @@ void vp9_build_interintra_16x16_predictors_mb(MACROBLOCKD *xd,
 void vp9_build_interintra_16x16_predictors_mby(MACROBLOCKD *xd,
                                                unsigned char *ypred,
                                                int ystride) {
-  static const int scale_bits = 6;
   unsigned char intrapredictor[256];
-  int i, j;
   vp9_build_intra_predictors_internal(
       xd->dst.y_buffer, xd->dst.y_stride,
       intrapredictor, 16,
@@ -640,7 +638,6 @@ void vp9_build_interintra_16x16_predictors_mbuv(MACROBLOCKD *xd,
                                                 unsigned char *upred,
                                                 unsigned char *vpred,
                                                 int uvstride) {
-  int i, j;
   unsigned char uintrapredictor[64];
   unsigned char vintrapredictor[64];
   vp9_build_intra_predictors_internal(
index e4a24f2cb1e74fbb5748a53008e75b673eb3d969..f542acb3df3af170aa2b062e113d5d7ea8bfd5eb 100644 (file)
@@ -105,7 +105,7 @@ B_PREDICTION_MODE vp9_find_dominant_direction(
 #elif CONTEXT_PRED_REPLACEMENTS == 0
 B_PREDICTION_MODE vp9_find_dominant_direction(
     unsigned char *ptr, int stride, int n) {
-  int g[8], i, imin, imin2, imax;
+  int g[8], i, imin, imax;
   g[0] = find_grad_measure(ptr, stride, n, 4,  1, 0);
   g[1] = find_grad_measure(ptr, stride, n, 4,  2, 1);
   g[2] = find_grad_measure(ptr, stride, n, 4,  1, 1);
index 810b693714bbc439e68635dc8a2e8ab80bbe05b6..1c0ce16e172785fa84fe8077579f49452a5dc2ba 100644 (file)
@@ -120,6 +120,12 @@ specialize vp9_recon_mb
 prototype void vp9_recon_mby "struct macroblockd *x"
 specialize vp9_recon_mby
 
+prototype void vp9_recon_mby_s "struct macroblockd *x, unsigned char *dst"
+specialize vp9_recon_mby_s
+
+prototype void vp9_recon_mbuv_s "struct macroblockd *x, unsigned char *udst, unsigned char *vdst"
+specialize void vp9_recon_mbuv_s
+
 prototype void vp9_build_intra_predictors_mby_s "struct macroblockd *x"
 specialize vp9_build_intra_predictors_mby_s
 
index c3962318feb8ffdce7374a6e0f30ba571104e146..8ea0f09ed89c89d95714b12ff974674722aaa354 100644 (file)
@@ -546,7 +546,6 @@ static void decode_4x4(VP9D_COMP *pbi, MACROBLOCKD *xd,
 #if CONFIG_SUPERBLOCKS
 static void decode_16x16_sb(VP9D_COMP *pbi, MACROBLOCKD *xd,
                             BOOL_DECODER* const bc, int n) {
-  BLOCKD *b = &xd->block[24];
   int x_idx = n & 1, y_idx = n >> 1;
   TX_TYPE tx_type = get_tx_type_16x16(xd, &xd->block[0]);
   if (tx_type != DCT_DCT) {
@@ -579,14 +578,9 @@ static void decode_8x8_sb(VP9D_COMP *pbi, MACROBLOCKD *xd,
     int i;
     for (i = 0; i < 4; i++) {
       int ib = vp9_i8x8_block[i];
-      const int iblock[4] = {0, 1, 4, 5};
-      const int ioffset[4] = {0, 1, 0, 1};
       int idx = (ib & 0x02) ? (ib + 2) : ib;
-      int i8x8mode = -1;
       short *q  = xd->block[idx].qcoeff;
       short *dq = xd->block[0].dequant;
-      unsigned char *pre = xd->block[ib].predictor;
-      unsigned char *dst = *(xd->block[ib].base_dst) + xd->block[ib].dst;
       int stride = xd->dst.y_stride;
       BLOCKD *b = &xd->block[ib];
       tx_type = get_tx_type_8x8(xd, &xd->block[ib]);
index f84e30e81ed5bc63c503fff1d8975295a7e5062f..72ef52b5a67aad7e42d8c68b12d59310765df598 100644 (file)
@@ -10,6 +10,9 @@
 
 #include "vp9_rtcd.h"
 #include "vp9/common/vp9_blockd.h"
+#if CONFIG_LOSSLESS
+#include "vp9/decoder/vp9_dequantize.h"
+#endif
 
 void vp9_dequant_dc_idct_add_y_block_c(short *q, const short *dq,
                                        unsigned char *pre,
index 2b11ed412c2c0942859393a050431853e1341c06..0adaeee0a14d6ee008b1bef942a320dbd3cb60b2 100644 (file)
@@ -2206,7 +2206,8 @@ void vp9_pack_bitstream(VP9_COMP *cpi, unsigned char *dest,
 #endif
     if (pc->mcomp_filter_type == SWITCHABLE)
       update_switchable_interp_probs(cpi, &header_bc);
-#if CONFIG_COMP_INTERINTRA_PRED
+
+    #if CONFIG_COMP_INTERINTRA_PRED
     if (pc->use_interintra) {
       vp9_cond_prob_update(&header_bc,
                            &pc->fc.interintra_prob,
index ab32fa12ccc786614574a0e76f3301669e7a1711..8ef442ae6604bf2272c517f53045268c0c0cf639 100644 (file)
@@ -399,11 +399,12 @@ static unsigned int pick_best_mv_ref(MACROBLOCK *x,
   for (i = 1; i < 4; ++i) {
     // If we see a 0,0 reference vector for a second time we have reached
     // the end of the list of valid candidate vectors.
-    if (!mv_ref_list[i].as_int)
+    if (!mv_ref_list[i].as_int) {
       if (zero_seen)
         break;
       else
         zero_seen = TRUE;
+    }
 
     // Check for cases where the reference choice would give rise to an
     // uncodable/out of range residual for row or col.
index 44d2d68fc788507d7757186199376b8616d55b60..8180d16de3a7bca86aa43f2b685486df7b784992 100644 (file)
@@ -577,8 +577,8 @@ typedef struct VP9_COMP {
   int mbsplit_count[VP9_NUMMBSPLITS];
   int y_uv_mode_count[VP9_YMODES][VP9_UV_MODES];
 #if CONFIG_COMP_INTERINTRA_PRED
-  int interintra_count[2];
-  int interintra_select_count[2];
+  unsigned int interintra_count[2];
+  unsigned int interintra_select_count[2];
 #endif
 
   nmv_context_counts NMVcount;
index 5a5e8a62d036a0f2bd8aedbfda32584cbbf15f59..84cff6edad2ec6c20f686efca50f7c65fcb94e9b 100644 (file)
@@ -3366,7 +3366,7 @@ static void rd_pick_inter_mode(VP9_COMP *cpi, MACROBLOCK *x,
   int distortion_uv = INT_MAX;
   int64_t best_yrd = INT64_MAX;
 #if CONFIG_PRED_FILTER
-  int best_filter_state;
+  int best_filter_state = 0;
 #endif
   int switchable_filter_index = 0;
 
@@ -3882,7 +3882,7 @@ static void rd_pick_inter_mode(VP9_COMP *cpi, MACROBLOCK *x,
     }
 #if CONFIG_COMP_INTERINTRA_PRED
     if ((mbmi->ref_frame == INTRA_FRAME) &&
-        (this_mode >= DC_PRED && this_mode <= TM_PRED) &&
+        (this_mode <= TM_PRED) &&
         (this_rd < best_intra16_rd)) {
       best_intra16_rd = this_rd;
       best_intra16_mode = this_mode;
@@ -4179,7 +4179,7 @@ void vp9_rd_pick_intra_mode(VP9_COMP *cpi, MACROBLOCK *x,
   int rate4x4d, dist4x4d;
 #endif
   int rate4x4, rate16x16 = 0, rateuv, rateuv8x8;
-  int dist4x4, dist16x16, distuv, distuv8x8;
+  int dist4x4 = 0, dist16x16 = 0, distuv = 0, distuv8x8 = 0;
   int rate;
   int rate4x4_tokenonly = 0;
   int rate16x16_tokenonly = 0;
@@ -4191,7 +4191,7 @@ void vp9_rd_pick_intra_mode(VP9_COMP *cpi, MACROBLOCK *x,
   int mode8x8[2][4];
   int dist;
   int modeuv, modeuv8x8, uv_intra_skippable, uv_intra_skippable_8x8;
-  int y_intra16x16_skippable;
+  int y_intra16x16_skippable = 0;
   int64_t txfm_cache[NB_TXFM_MODES];
   TX_SIZE txfm_size_16x16;
   int i;
@@ -4356,16 +4356,17 @@ int64_t vp9_rd_pick_inter_mode_sb(VP9_COMP *cpi, MACROBLOCK *x,
   int64_t best_pred_diff[NB_PREDICTION_TYPES];
   int64_t best_pred_rd[NB_PREDICTION_TYPES];
   MB_MODE_INFO best_mbmode;
-  int mode_index, best_mode_index;
+  int mode_index, best_mode_index = 0;
   unsigned int ref_costs[MAX_REF_FRAMES];
 #if CONFIG_COMP_INTERINTRA_PRED
   int is_best_interintra = 0;
   int64_t best_intra16_rd = INT64_MAX;
   int best_intra16_mode = DC_PRED, best_intra16_uv_mode = DC_PRED;
 #endif
-  int rate_uv_4x4, rate_uv_8x8, rate_uv_tokenonly_4x4, rate_uv_tokenonly_8x8;
-  int dist_uv_4x4, dist_uv_8x8, uv_skip_4x4, uv_skip_8x8;
-  MB_PREDICTION_MODE mode_uv_4x4, mode_uv_8x8;
+  int rate_uv_4x4 = 0, rate_uv_8x8 = 0, rate_uv_tokenonly_4x4 = 0,
+      rate_uv_tokenonly_8x8 = 0;
+  int dist_uv_4x4 = 0, dist_uv_8x8 = 0, uv_skip_4x4 = 0, uv_skip_8x8 = 0;
+  MB_PREDICTION_MODE mode_uv_4x4 = NEARESTMV, mode_uv_8x8 = NEARESTMV;
 
   x->skip = 0;
   xd->mode_info_context->mbmi.segment_id = segment_id;