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 b086666..51a5676 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 96ebbf9..264668a 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;