Remove clang compiler warnings for Android: -Wno-unused-parameter

Summary: att

Reviewed By: gkmhub

Differential Revision: D8972835

fbshipit-source-id: c8b893eefec6ccb5d8506c959bcf5c3f43701e81

lib/fb/src/main/cpp/include/fb/fbjni/CoreClasses-inl.h

@@ -1,10 +1,10 @@
 /*
  *  Copyright (c) 2015-present, Facebook, Inc.
  *
- * This source code is licensed under the MIT license found in the
+ *  This source code is licensed under the MIT license found in the LICENSE
- * LICENSE file in the root directory of this source tree.
+ *  file in the root directory of this source tree.
+ *
  */
-
 #pragma once
 
 #include <string.h>
@@ -509,6 +509,8 @@ class PinnedCriticalAlloc {
       T** elements,
       size_t* size,
       jboolean* isCopy) {
+    (void)start;
+    (void)length;
     const auto env = internal::getEnv();
     *elements = static_cast<T*>(env->GetPrimitiveArrayCritical(array.get(), isCopy));
     FACEBOOK_JNI_THROW_EXCEPTION_IF(!elements);
@@ -520,6 +522,8 @@ class PinnedCriticalAlloc {
       jint start,
       jint size,
       jint mode) {
+    (void)start;
+    (void)size;
     const auto env = internal::getEnv();
     env->ReleasePrimitiveArrayCritical(array.get(), elements, mode);
   }

lib/fb/src/main/cpp/include/fb/fbjni/Hybrid.h

@@ -1,10 +1,10 @@
 /*
  *  Copyright (c) 2015-present, Facebook, Inc.
  *
- * This source code is licensed under the MIT license found in the
+ *  This source code is licensed under the MIT license found in the LICENSE
- * LICENSE file in the root directory of this source tree.
+ *  file in the root directory of this source tree.
+ *
  */
-
 #pragma once
 
 #include <memory>
@@ -188,7 +188,9 @@ public:
   // particular java class) to be hoisted to a common function.  If
   // mapException returns, then the std::exception will be translated
   // to Java.
-  static void mapException(const std::exception& ex) {}
+  static void mapException(const std::exception& ex) {
+    (void)ex;
+  }
 };
 
 template <typename T, typename B>

lib/fb/src/main/cpp/include/fb/fbjni/Meta-inl.h

@@ -1,10 +1,10 @@
 /*
  *  Copyright (c) 2015-present, Facebook, Inc.
  *
- * This source code is licensed under the MIT license found in the
+ *  This source code is licensed under the MIT license found in the LICENSE
- * LICENSE file in the root directory of this source tree.
+ *  file in the root directory of this source tree.
+ *
  */
-
 #pragma once
 
 #include <jni.h>
@@ -53,6 +53,7 @@ struct ArgsArraySetter<idx, Arg, Args...> {
 template <int idx>
 struct ArgsArraySetter<idx> {
   static void set(alias_ref<JArrayClass<jobject>::javaobject> array) {
+    (void)array;
   }
 };
 

lib/fb/src/main/cpp/include/fb/fbjni/References-inl.h

@@ -1,10 +1,10 @@
 /*
  *  Copyright (c) 2015-present, Facebook, Inc.
  *
- * This source code is licensed under the MIT license found in the
+ *  This source code is licensed under the MIT license found in the LICENSE
- * LICENSE file in the root directory of this source tree.
+ *  file in the root directory of this source tree.
+ *
  */
-
 #pragma once
 
 #include <new>
@@ -182,10 +182,11 @@ inline base_owned_ref<T, Alloc>::base_owned_ref() noexcept
   : base_owned_ref(nullptr)
 {}
 
-template<typename T, typename Alloc>
+template <typename T, typename Alloc>
-inline base_owned_ref<T, Alloc>::base_owned_ref(std::nullptr_t t) noexcept
+inline base_owned_ref<T, Alloc>::base_owned_ref(std::nullptr_t array) noexcept
-  : base_owned_ref(static_cast<javaobject>(nullptr))
+    : base_owned_ref(static_cast<javaobject>(nullptr)) {
-{}
+  (void)array;
+}
 
 template<typename T, typename Alloc>
 inline base_owned_ref<T, Alloc>::base_owned_ref(const base_owned_ref& other)

lib/fb/src/main/cpp/include/fb/fbjni/Registration-inl.h

@@ -1,10 +1,10 @@
 /*
  *  Copyright (c) 2015-present, Facebook, Inc.
  *
- * This source code is licensed under the MIT license found in the
+ *  This source code is licensed under the MIT license found in the LICENSE
- * LICENSE file in the root directory of this source tree.
+ *  file in the root directory of this source tree.
+ *
  */
-
 #pragma once
 
 #include "Exceptions.h"
@@ -106,8 +106,10 @@ inline NativeMethodWrapper* exceptionWrapJNIMethod(R (*)(alias_ref<C>, Args... a
 
 // registration wrappers for non-static methods, with autoconvertion of arguments.
 
-template<typename M, M method, typename C, typename... Args>
+template <typename M, M method, typename C, typename... Args>
-inline NativeMethodWrapper* exceptionWrapJNIMethod(void (C::*method0)(Args... args)) {
+inline NativeMethodWrapper* exceptionWrapJNIMethod(
+    void (C::*method0)(Args... args)) {
+  (void)method0;
   struct funcWrapper {
     JNI_ENTRY_POINT static void call(JNIEnv* env, jobject obj,
                                      typename Convert<typename std::decay<Args>::type>::jniType... args) {