]> granicus.if.org Git - libvpx/commitdiff
Add protection for codec interface name change.
authorFrank Galligan <fgalligan@google.com>
Mon, 16 Mar 2015 19:59:07 +0000 (12:59 -0700)
committerFrank Galligan <fgalligan@google.com>
Tue, 17 Mar 2015 20:46:22 +0000 (13:46 -0700)
Change-Id: I9101cb1132aa580fc32fd47e8ac9d91d5b09e963

test/encode_test_driver.cc

index f3d8e31daf76765e6692f7d698c5be5bfbef5bd9..bdd71c6fe3a82bf447828f3da3cc0e3b92eec5e8 100644 (file)
@@ -29,18 +29,26 @@ void Encoder::InitEncoder(VideoSource *video) {
     cfg_.g_timebase = video->timebase();
     cfg_.rc_twopass_stats_in = stats_->buf();
 
-    // Default to 1 thread and 1 tile column.
+    // Default to 1 thread.
     cfg_.g_threads = 1;
     res = vpx_codec_enc_init(&encoder_, CodecInterface(), &cfg_,
                              init_flags_);
     ASSERT_EQ(VPX_CODEC_OK, res) << EncoderError();
 
-    std::string codec_name(encoder_.name);
-    if (codec_name.find("WebM Project VP9 Encoder") != std::string::npos) {
+#if CONFIG_VP9_ENCODER
+    if (CodecInterface() == &vpx_codec_vp9_cx_algo) {
+      // Default to 1 tile column for VP9.
       const int log2_tile_columns = 0;
       res = vpx_codec_control_(&encoder_, VP9E_SET_TILE_COLUMNS,
                                log2_tile_columns);
       ASSERT_EQ(VPX_CODEC_OK, res) << EncoderError();
+    } else
+#endif
+    {
+#if CONFIG_VP8_ENCODER
+      ASSERT_EQ(&vpx_codec_vp8_cx_algo, CodecInterface())
+          << "Unknown Codec Interface";
+#endif
     }
   }
 }