wangle: make new pragma use portable to older clang
authorJim Meyering <meyering@fb.com>
Sat, 14 Feb 2015 00:37:37 +0000 (16:37 -0800)
committerAlecs King <int@fb.com>
Tue, 3 Mar 2015 03:20:56 +0000 (19:20 -0800)
Summary:
* folly/wangle/channel/test/MockChannelHandler.h:
Avoiding a new warning that rendered this file uncompilable
by clang-3.5 and older.  Add cpp conditionals to protect
against that.

Test Plan:
Ensure it works with clang-3.4, 3.5 and clang:dev.
I.e., ensure that this prints PASS at the end.

for i in '' 3.5 dev; do
test -n "$i" && i=--with-project-version=clang:$i
fbconfig -r --clang $i folly/wangle/channel/test && fbmake runtests_dbgo
done && echo PASS

Reviewed By: meyering@fb.com

Subscribers: mpawlowski, fugalh, mathieubaudet, folly-diffs@, jsedgwick, yfeldblum

FB internal diff: D1849474

Tasks: 6244745

Blame Revision: D1848330
@build-break

folly/wangle/channel/test/MockChannelHandler.h

index 930867044613ce045ddeede005eb99d0531bf011..c611bd56ae7f7c14c78dfdd54fe5e23b6c582965 100644 (file)
@@ -30,8 +30,10 @@ class MockChannelHandler : public ChannelHandler<Rin, Rout, Win, Wout> {
   MockChannelHandler(MockChannelHandler&&) = default;
 
 #ifdef __clang__
-#pragma clang diagnostic push
-#pragma clang diagnostic ignored "-Winconsistent-missing-override"
+# 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&));