From 7826a2b6d6f0aff10f478ee420a41615c84e12fd Mon Sep 17 00:00:00 2001
From: Eric Hughes <eric.hughes@tiledb.com>
Date: Thu, 5 Oct 2023 16:03:12 -0600
Subject: [PATCH] Restored argument names that were apparently, at one point,
 unused but are now very much used. clang-format

---
 tiledb/sm/misc/parallel_functions.h          | 2 +-
 tiledb/sm/query/test/unit_query_condition.cc | 2 +-
 tiledb/sm/rtree/rtree.cc                     | 4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/tiledb/sm/misc/parallel_functions.h b/tiledb/sm/misc/parallel_functions.h
index 0b9d3056126..a30b6423468 100644
--- a/tiledb/sm/misc/parallel_functions.h
+++ b/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());
diff --git a/tiledb/sm/query/test/unit_query_condition.cc b/tiledb/sm/query/test/unit_query_condition.cc
index 7f28ccbfade..6b5f82f891a 100644
--- a/tiledb/sm/query/test/unit_query_condition.cc
+++ b/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;
diff --git a/tiledb/sm/rtree/rtree.cc b/tiledb/sm/rtree/rtree.cc
index 458fda569d5..8578636c371 100644
--- a/tiledb/sm/rtree/rtree.cc
+++ b/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();