shithub: libvpx

Info  •  Files  •  Log  •  Branches

Remove unneeded variable declaration

This patch fixed the following warning:
src\third_party\libvpx\source\libvpx\vp9\encoder\vp9_pickmode.c(1607) :
warning C6246: Local declaration of 'this_mode' hides declaration of the
same name in outer scope.

Change-Id: I1d93c4a47a13cb13089fec5bd61e8b58e6cd8d58

--- a/vp9/encoder/vp9_pickmode.c

+++ b/vp9/encoder/vp9_pickmode.c

@@ -1594,7 +1594,6 @@

   if (cpi->sf.adaptive_rd_thresh) {

     THR_MODES best_mode_idx = mode_idx[best_ref_frame][mode_offset(mbmi->mode)];

-    PREDICTION_MODE this_mode;

     if (best_ref_frame == INTRA_FRAME) {

       // Only consider the modes that are included in the intra_mode_list.

@@ -1604,12 +1603,12 @@

       // TODO(yunqingwang): Check intra mode mask and only update freq_fact

       // for those valid modes.

       for (i = 0; i < intra_modes; i++) {

-        PREDICTION_MODE this_mode = intra_mode_list[i];

         update_thresh_freq_fact(cpi, tile_data, bsize, INTRA_FRAME,

-                                best_mode_idx, this_mode);

+                                best_mode_idx, intra_mode_list[i]);

       }

     } else {

       for (ref_frame = LAST_FRAME; ref_frame <= GOLDEN_FRAME; ++ref_frame) {

+        PREDICTION_MODE this_mode;

         if (best_ref_frame != ref_frame) continue;

         for (this_mode = NEARESTMV; this_mode <= NEWMV; ++this_mode) {

           update_thresh_freq_fact(cpi, tile_data, bsize, ref_frame,