]> granicus.if.org Git - libvpx/commitdiff
test/svc_*_test: fix SetConfig() signature
authorJames Zern <jzern@google.com>
Sat, 8 Dec 2018 08:43:31 +0000 (00:43 -0800)
committerJames Zern <jzern@google.com>
Sun, 9 Dec 2018 07:17:31 +0000 (07:17 +0000)
make the parameter constant to match the base class and mark the
function virtual. virtual is used to match the rest of the code base,
but now that c++11 is required all such functions could be changed to
override.

Change-Id: I551a05bbd9d05a9eddb653f42eaad68880c88141

test/svc_datarate_test.cc
test/svc_end_to_end_test.cc

index b086666723890dbfbcc5d6d5e0d3c37684680469..51a56764c0805d64a5b0b936650a16394cf5d064 100644 (file)
@@ -537,7 +537,7 @@ class DatarateOnePassCbrSvc : public ::svc_test::OnePassCbrSvc {
   bool denoiser_enable_layers_;
 
  private:
-  void SetConfig(int num_temporal_layer) {
+  virtual void SetConfig(const int num_temporal_layer) {
     cfg_.rc_end_usage = VPX_CBR;
     cfg_.g_lag_in_frames = 0;
     cfg_.g_error_resilient = 1;
index 96ebbf9d11dacd4d1d41018a2b20e949a438edba..264668ab4c460cdd027d296a4511a23ed004a0f2 100644 (file)
@@ -123,7 +123,7 @@ class SyncFrameOnePassCbrSvc : public ::svc_test::OnePassCbrSvc,
   vpx_svc_spatial_layer_sync_t svc_layer_sync_;
 
  private:
-  void SetConfig(int num_temporal_layer) {
+  virtual void SetConfig(const int num_temporal_layer) {
     cfg_.rc_buf_initial_sz = 500;
     cfg_.rc_buf_optimal_sz = 500;
     cfg_.rc_buf_sz = 1000;