]> granicus.if.org Git - libvpx/commitdiff
ethread_test: Remove vp10 as test parameter.
authorAlex Converse <aconverse@google.com>
Tue, 28 Jun 2016 21:21:13 +0000 (14:21 -0700)
committerAlex Converse <aconverse@google.com>
Tue, 28 Jun 2016 21:32:15 +0000 (14:32 -0700)
Change-Id: I043418cde5a2562520ff37cdf81436abc2c9821a

test/vp9_ethread_test.cc

index 78f1ec3ff9c3cb9879276d6a584645dd87d76cf5..aa84a9e4fe80f41bde15b354478b11a0cb90017a 100644 (file)
 namespace {
 class VPxEncoderThreadTest
     : public ::libvpx_test::EncoderTest,
-      public ::libvpx_test::CodecTestWith3Params<libvpx_test::TestMode,
-                                                 int, int> {
+      public ::libvpx_test::CodecTestWith2Params<libvpx_test::TestMode, int> {
  protected:
   VPxEncoderThreadTest()
       : EncoderTest(GET_PARAM(0)),
         encoder_initialized_(false),
         encoding_mode_(GET_PARAM(1)),
-        set_cpu_used_(GET_PARAM(2)),
-        vp10_(GET_PARAM(3)) {
+        set_cpu_used_(GET_PARAM(2)) {
     init_flags_ = VPX_CODEC_USE_PSNR;
     vpx_codec_dec_cfg_t cfg = vpx_codec_dec_cfg_t();
     cfg.w = 1280;
@@ -74,9 +72,9 @@ class VPxEncoderThreadTest
   virtual void PreEncodeFrameHook(::libvpx_test::VideoSource * /*video*/,
                                   ::libvpx_test::Encoder *encoder) {
     if (!encoder_initialized_) {
-#if CONFIG_EXT_TILE
+#if CONFIG_VP10 && CONFIG_EXT_TILE
       encoder->Control(VP9E_SET_TILE_COLUMNS, 1);
-      if (vp10_) {
+      if (codec_ == &libvpx_test::kVP10) {
         // TODO(geza): Start using multiple tile rows when the multi-threaded
         // encoder can handle them
         encoder->Control(VP9E_SET_TILE_ROWS, 32);
@@ -87,7 +85,7 @@ class VPxEncoderThreadTest
       // Encode 4 tile columns.
       encoder->Control(VP9E_SET_TILE_COLUMNS, 2);
       encoder->Control(VP9E_SET_TILE_ROWS, 0);
-#endif  // CONFIG_EXT_TILE
+#endif  // CONFIG_VP10 && CONFIG_EXT_TILE
       encoder->Control(VP8E_SET_CPUUSED, set_cpu_used_);
       if (encoding_mode_ != ::libvpx_test::kRealTime) {
         encoder->Control(VP8E_SET_ENABLEAUTOALTREF, 1);
@@ -169,8 +167,6 @@ class VPxEncoderThreadTest
   std::vector<size_t> size_enc_;
   std::vector<std::string> md5_enc_;
   std::vector<std::string> md5_dec_;
-
-  bool vp10_;
 };
 
 TEST_P(VPxEncoderThreadTest, EncoderResultTest) {
@@ -187,15 +183,15 @@ VP9_INSTANTIATE_TEST_CASE(
     VPxEncoderThreadTest,
     ::testing::Values(::libvpx_test::kTwoPassGood, ::libvpx_test::kOnePassGood,
                       ::libvpx_test::kRealTime),
-    ::testing::Range(1, 9), ::testing::Values(0));
+    ::testing::Range(1, 9));
 
 VP10_INSTANTIATE_TEST_CASE(
     VPxEncoderThreadTest,
     ::testing::Values(::libvpx_test::kTwoPassGood, ::libvpx_test::kOnePassGood),
-    ::testing::Range(3, 9), ::testing::Values(1));
+    ::testing::Range(3, 9));
 
 VP10_INSTANTIATE_TEST_CASE(
     VPxEncoderThreadTestLarge,
     ::testing::Values(::libvpx_test::kTwoPassGood, ::libvpx_test::kOnePassGood),
-    ::testing::Range(1, 3), ::testing::Values(1));
+    ::testing::Range(1, 3));
 }  // namespace