add preprocessor to hide unused functions
authorBeny Luo <benyluo@fb.com>
Mon, 23 Feb 2015 15:12:29 +0000 (07:12 -0800)
committerAlecs King <int@fb.com>
Tue, 3 Mar 2015 03:28:27 +0000 (19:28 -0800)
Summary:
encodeVarintToIOBuf and decodeVarintFromCursor are not used when
FOLLY_HAVE_LIBLZMA and FOLLY_HAVE_LIBLZ4 are false, which will trigger
-Werror,-Wunused-function on compile on mobile.

Test Plan: unit test

Reviewed By: seanc@fb.com

Subscribers: meisner, trunkagent, seanc, folly-diffs@, yfeldblum

FB internal diff: D1864016

Signature: t1:1864016:1424702813:75535ea92ac16385ce5c272d93bba507ccbb67a1

folly/io/Compression.cpp

index ab0ee5a0719bff6e5aea2bb393be22f2978305e6..9425f98f278c69595380b575ce7723272b99829d 100644 (file)
@@ -148,6 +148,8 @@ std::unique_ptr<IOBuf> NoCompressionCodec::doUncompress(
   return data->clone();
 }
 
+#if (FOLLY_HAVE_LIBLZ4 || FOLLY_HAVE_LIBLZMA)
+
 namespace {
 
 void encodeVarintToIOBuf(uint64_t val, folly::IOBuf* out) {
@@ -173,6 +175,8 @@ inline uint64_t decodeVarintFromCursor(folly::io::Cursor& cursor) {
 
 }  // namespace
 
+#endif  // FOLLY_HAVE_LIBLZ4 || FOLLY_HAVE_LIBLZMA
+
 #if FOLLY_HAVE_LIBLZ4
 
 /**