Restored argument names that were apparently, at one point, unused bu…

…t are now very much used.

clang-format

tiledb/sm/misc/parallel_functions.h

@@ -272,7 +272,7 @@ Status parallel_for(
   // Wait for all instances of `execute_subrange` to complete.
   // This is ignoring the wait status as we use failed_exception for propagating
   // the tasks exceptions.
-  (void) tp->wait_all(tasks);
+  (void)tp->wait_all(tasks);
 
   if (failed_exception.has_value()) {
     std::rethrow_exception(failed_exception.value());

tiledb/sm/query/test/unit_query_condition.cc

@@ -1595,7 +1595,7 @@ void test_apply<char*>(const Datatype type, bool var_size, bool nullable) {
  * Non-specialized template type for `test_apply`.
  */
 template <typename T>
-void test_apply(const Datatype type, bool, bool) {
+void test_apply(const Datatype type, bool var_size, bool nullable) {
   const std::string field_name = "foo";
   const uint64_t cells = 10;
   const T fill_value = 3;

tiledb/sm/rtree/rtree.cc

@@ -361,12 +361,12 @@ void RTree::deserialize_v1_v4(
   deserialized_buffer_size_ = deserializer.size();
 
   // For backwards compatibility, ignored
-  (void) deserializer.read<unsigned>();
+  (void)deserializer.read<unsigned>();
 
   fanout_ = deserializer.read<unsigned>();
 
   // For backwards compatibility, ignored
-  (void) deserializer.read<uint8_t>();
+  (void)deserializer.read<uint8_t>();
 
   auto level_num = deserializer.read<unsigned>();
   levels_.clear();