From: James Zern <jzern@google.com>
Date: Sat, 8 Dec 2018 08:43:31 +0000 (-0800)
Subject: test/svc_*_test: fix SetConfig() signature
X-Git-Tag: v1.8.0~85^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=70391e8fd69cb72b5babd7fbb9e8927ecdbefb6b;p=libvpx

test/svc_*_test: fix SetConfig() signature

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
---

diff --git a/test/svc_datarate_test.cc b/test/svc_datarate_test.cc
index b08666672..51a56764c 100644
--- a/test/svc_datarate_test.cc
+++ b/test/svc_datarate_test.cc
@@ -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;
diff --git a/test/svc_end_to_end_test.cc b/test/svc_end_to_end_test.cc
index 96ebbf9d1..264668ab4 100644
--- a/test/svc_end_to_end_test.cc
+++ b/test/svc_end_to_end_test.cc
@@ -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;