Remove -Wno-inconsistent-missing-override suppression in folly

Summary: It is fixed in gmock. No need to suppress it in sources.

Reviewed By: @meyering

Differential Revision: D2181564

diff --git a/folly/wangle/channel/test/MockHandler.h b/folly/wangle/channel/test/MockHandler.h
index 5a476646293d307861bc2afa41d9a461e2bcbbb5..eb5f44ced434b5031f99a93f72411e8da3d8ea26 100644 (file)
--- a/folly/wangle/channel/test/MockHandler.h
+++ b/folly/wangle/channel/test/MockHandler.h
@@ -29,13 +29,6 @@ class MockHandler : public Handler<Rin, Rout, Win, Wout> {
   MockHandler() = default;
   MockHandler(MockHandler&&) = default;
 
   MockHandler() = default;
   MockHandler(MockHandler&&) = default;
 

-#ifdef __clang__
-# pragma clang diagnostic push
-# if __clang_major__ > 3 || __clang_minor__ >= 6
-#  pragma clang diagnostic ignored "-Winconsistent-missing-override"
-# endif
-#endif
-

   MOCK_METHOD2_T(read_, void(Context*, Rin&));
   MOCK_METHOD1_T(readEOF, void(Context*));
   MOCK_METHOD2_T(readException, void(Context*, exception_wrapper));
   MOCK_METHOD2_T(read_, void(Context*, Rin&));
   MOCK_METHOD1_T(readEOF, void(Context*));
   MOCK_METHOD2_T(readException, void(Context*, exception_wrapper));


@@ -48,10 +41,6 @@ class MockHandler : public Handler<Rin, Rout, Win, Wout> {
   MOCK_METHOD1_T(detachPipeline, void(Context*));
   MOCK_METHOD1_T(detachTransport, void(Context*));
 
   MOCK_METHOD1_T(detachPipeline, void(Context*));
   MOCK_METHOD1_T(detachTransport, void(Context*));
 

-#ifdef __clang__
-#pragma clang diagnostic pop
-#endif
-

   void read(Context* ctx, Rin msg) override {
     read_(ctx, msg);
   }
   void read(Context* ctx, Rin msg) override {
     read_(ctx, msg);
   }