]> granicus.if.org Git - libvpx/commitdiff
vp9-svc: Fix to allowed value of max_consec_drop.
authorMarco Paniconi <marpan@google.com>
Tue, 29 May 2018 02:32:01 +0000 (19:32 -0700)
committerMarco Paniconi <marpan@google.com>
Tue, 29 May 2018 04:13:10 +0000 (21:13 -0700)
For the max_consec_drop parameter in svc frame drop:
since passing value 0 in the control would completely
disable the dropper, only allow for values >= 1 to be set.

Change-Id: I6b74ec9cc08a638fa571d6246a021dab9c811d14

vp9/vp9_cx_iface.c

index 5303a71de47810bdec6ce49ad4860d4f23c843a8..2758314fb2e7c21037e353d3699cf74b8086421c 100644 (file)
@@ -1538,7 +1538,8 @@ static vpx_codec_err_t ctrl_set_svc_frame_drop_layer(vpx_codec_alg_priv_t *ctx,
   cpi->svc.framedrop_mode = data->framedrop_mode;
   for (sl = 0; sl < cpi->svc.number_spatial_layers; ++sl)
     cpi->svc.framedrop_thresh[sl] = data->framedrop_thresh[sl];
-  cpi->svc.max_consec_drop = data->max_consec_drop;
+  // Don't allow max_consec_drop values below 1.
+  cpi->svc.max_consec_drop = VPXMAX(1, data->max_consec_drop);
   return VPX_CODEC_OK;
 }