commit 606098cac8ea2704314a5981932740795da3ff47
author Wouter van Oortmerssen <wvo@google.com> Mon Oct 17 13:06:11 2016 -0700
committer Wouter van Oortmerssen <wvo@google.com> Mon Oct 17 13:06:11 2016 -0700
tree 2ea57dc59d94321606822094dc6e7445d873a899
parent b5c622762bb47e3566f5e46dfba0b3b7fac32510

Fixed unused variable warnings, and made sure they are on.

Tested on Linux.

Change-Id: Ie18d6857f9ae8b1c3953a116203b4a8bb70a2ede

CMakeLists.txt

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 1ef3372..835aa3e 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -100,7 +100,7 @@
 # source_group(Tests FILES ${FlatBuffers_Tests_SRCS})
 
 if(EXISTS "${CMAKE_TOOLCHAIN_FILE}")
-  # do not apply any global settings if the toolchain 
+  # do not apply any global settings if the toolchain
   # is being configured externally
 elseif(APPLE)
   set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -stdlib=libc++")
@@ -117,7 +117,8 @@
     "${CMAKE_CXX_FLAGS} -Wall -pedantic -Werror -Wextra -Werror=shadow")
   if (GCC_VERSION VERSION_GREATER 4.4)
     set(CMAKE_CXX_FLAGS
-      "${CMAKE_CXX_FLAGS} -Wunused-result -Werror=unused-result")
+      "${CMAKE_CXX_FLAGS} -Wunused-result -Werror=unused-result \
+                          -Wunused-parameter -Werror=unused-parameter")
   endif()
 
   # Certain platforms such as ARM do not use signed chars by default

tests/test.cpp

diff --git a/tests/test.cpp b/tests/test.cpp
index f2854c6..34a556e 100644
--- a/tests/test.cpp
+++ b/tests/test.cpp
@@ -319,9 +319,12 @@
   // and such.
   auto resolver = flatbuffers::resolver_function_t([](void **pointer_adr,
                                                flatbuffers::hash_value_t hash) {
+    (void)pointer_adr;
+    (void)hash;
     // Don't actually do anything, leave variable null.
   });
   auto rehasher = flatbuffers::rehasher_function_t([](void *pointer) {
+    (void)pointer;
     return 0;
   });