Enable -Wunused-variables
authorChristopher Dykes <cdykes@fb.com>
Tue, 19 Sep 2017 05:54:48 +0000 (22:54 -0700)
committerFacebook Github Bot <facebook-github-bot@users.noreply.github.com>
Tue, 19 Sep 2017 06:10:22 +0000 (23:10 -0700)
commit94b8816bbdca38914030842554cc955caee59063
tree3a403a86680ed2990a54c34148f7fdc136a4b159
parentf3bce4241d7d3f1a80cc9a5391e7012af5f67f5a
Enable -Wunused-variables

Summary:
Only for clang for the moment, as more work is needed for GCC.
MSVC builds have always had unused variable warnings enabled, I just hadn't gotten around to cleaning up all of the tests for it to be able to enable it.

Reviewed By: yfeldblum

Differential Revision: D5827840

fbshipit-source-id: ab503b5791fcc58d685b8327179b810880c5dea7
20 files changed:
folly/RWSpinLock.h
folly/concurrency/detail/ConcurrentHashMap-detail.h
folly/concurrency/test/ConcurrentHashMapTest.cpp
folly/executors/IOThreadPoolExecutor.cpp
folly/experimental/bser/test/BserTest.cpp
folly/experimental/exception_tracer/test/ExceptionTracerBenchmark.cpp
folly/experimental/logging/LoggerDB.cpp
folly/experimental/test/FutureDAGTest.cpp
folly/futures/test/TestExecutor.cpp
folly/gen/test/BaseBenchmark.cpp
folly/gen/test/ParallelBenchmark.cpp
folly/gen/test/StringTest.cpp
folly/test/FBVectorTestBenchmarks.cpp.h
folly/test/IPAddressBenchmark.cpp
folly/test/OptionalCoroutinesTest.cpp
folly/test/RangeTest.cpp
folly/test/ScopeGuardTest.cpp
folly/test/UncaughtExceptionsTest.cpp
folly/test/function_benchmark/main.cpp
folly/test/stl_tests/StlVectorTest.cpp