From: James Zern <jzern@google.com>
Date: Wed, 19 Mar 2014 02:25:31 +0000 (-0700)
Subject: intrapred_test: fix inheritance
X-Git-Tag: v1.4.0~2021^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9d7080e9f6bba3e840a98e5d9b73c538e30b5f05;p=libvpx

intrapred_test: fix inheritance

inheritance should be public; also correct placement of ClearSystemState
as the base class doesn't inherit from testing

Change-Id: I0f41330fccc62a70b8dd40d66bbd829b9d98cf84
---

diff --git a/test/intrapred_test.cc b/test/intrapred_test.cc
index b28f5fbe6..cefe192fb 100644
--- a/test/intrapred_test.cc
+++ b/test/intrapred_test.cc
@@ -26,11 +26,7 @@ using libvpx_test::ACMRandom;
 
 class IntraPredBase {
  public:
-  virtual ~IntraPredBase() {}
-
-  virtual void TearDown() {
-    libvpx_test::ClearSystemState();
-  }
+  virtual ~IntraPredBase() { libvpx_test::ClearSystemState(); }
 
  protected:
   void SetupMacroblock(MACROBLOCKD *mbptr,
@@ -227,8 +223,9 @@ typedef void (*intra_pred_y_fn_t)(MACROBLOCKD *x,
                                   uint8_t *ypred_ptr,
                                   int y_stride);
 
-class IntraPredYTest : public ::testing::TestWithParam<intra_pred_y_fn_t>,
-    protected IntraPredBase {
+class IntraPredYTest
+    : public IntraPredBase,
+      public ::testing::TestWithParam<intra_pred_y_fn_t> {
  public:
   static void SetUpTestCase() {
     mb_ = reinterpret_cast<MACROBLOCKD*>(
@@ -308,8 +305,9 @@ typedef void (*intra_pred_uv_fn_t)(MACROBLOCKD *x,
                                    uint8_t *vpred_ptr,
                                    int pred_stride);
 
-class IntraPredUVTest : public ::testing::TestWithParam<intra_pred_uv_fn_t>,
-    protected IntraPredBase {
+class IntraPredUVTest
+    : public IntraPredBase,
+      public ::testing::TestWithParam<intra_pred_uv_fn_t> {
  public:
   static void SetUpTestCase() {
     mb_ = reinterpret_cast<MACROBLOCKD*>(