shithub: libvpx

Info  •  Files  •  Log  •  Branches

Removing redundant checks from vp9_change_config().

Both values are already checked as command line arguments:
RANGE_CHECK_HI(cfg, g_lag_in_frames, MAX_LAG_BUFFERS);
RANGE_CHECK_HI(extra_cfg, sharpness, 7);

Change-Id: I584798d587152d88dfd517c210054b466f4e5f8a

--- a/vp9/encoder/vp9_onyx_if.c

+++ b/vp9/encoder/vp9_onyx_if.c

@@ -861,11 +861,8 @@

   cm->display_width = cpi->oxcf.width;

   cm->display_height = cpi->oxcf.height;

-  // VP8 sharpness level mapping 0-7 (vs 0-10 in general VPx dialogs)

-  cpi->oxcf.sharpness = MIN(7, cpi->oxcf.sharpness);

+  cm->lf.sharpness_level = cpi->oxcf.sharpness;

-  cpi->common.lf.sharpness_level = cpi->oxcf.sharpness;

-

   if (cpi->initial_width) {

     // Increasing the size of the frame beyond the first seen frame, or some

     // otherwise signaled maximum size, is not supported.

@@ -883,10 +880,6 @@

   }

   cpi->speed = abs(cpi->oxcf.cpu_used);

-

-  // Limit on lag buffers as these are not currently dynamically allocated.

-  if (cpi->oxcf.lag_in_frames > MAX_LAG_BUFFERS)

-    cpi->oxcf.lag_in_frames = MAX_LAG_BUFFERS;

 #if CONFIG_MULTIPLE_ARF

   vp9_zero(cpi->alt_ref_source);