Update libCacheSim/dataStructure/minimalIncrementCBF.h

1a1a11a · Copilot · 1a1a11a · commit 34a052a7470c · 2025-06-22T15:02:12.000-04:00
Co-authored-by: Copilot &lt;175728472+Copilot@users.noreply.github.com&gt;
diff --git a/libCacheSim/bin/traceUtils/utils.cpp b/libCacheSim/bin/traceUtils/utils.cpp
@@ -13,7 +13,7 @@
 namespace utils {
 void *setup_mmap(const std::string &file_path, size_t *size) {
   int fd;
-  struct stat st{};
+  struct stat st {};
   void *mapped_file;
 
   // set up mmap region
diff --git a/libCacheSim/dataStructure/hash/xxh3.h b/libCacheSim/dataStructure/hash/xxh3.h
@@ -1742,9 +1742,11 @@ static XXH64_hash_t XXH3_mergeAccs(const xxh_u64* XXH_RESTRICT acc,
   return XXH3_avalanche(result64);
 }
 
-#define XXH3_INIT_ACC                                          \
-  {XXH_PRIME32_3, XXH_PRIME64_1, XXH_PRIME64_2, XXH_PRIME64_3, \
-   XXH_PRIME64_4, XXH_PRIME32_2, XXH_PRIME64_5, XXH_PRIME32_1}
+#define XXH3_INIT_ACC                                                          \
+  {                                                                            \
+    XXH_PRIME32_3, XXH_PRIME64_1, XXH_PRIME64_2, XXH_PRIME64_3, XXH_PRIME64_4, \
+        XXH_PRIME32_2, XXH_PRIME64_5, XXH_PRIME32_1                            \
+  }
 
 XXH_FORCE_INLINE XXH64_hash_t XXH3_hashLong_64b_internal(
     const xxh_u8* XXH_RESTRICT input, size_t len,
diff --git a/libCacheSim/dataStructure/hash/xxhash.h b/libCacheSim/dataStructure/hash/xxhash.h
@@ -434,7 +434,7 @@ struct XXH32_state_s {
   XXH32_hash_t memsize;
   XXH32_hash_t
       reserved; /* never read nor write, might be removed in a future version */
-}; /* typedef'd to XXH32_state_t */
+};              /* typedef'd to XXH32_state_t */
 
 #ifndef XXH_NO_LONG_LONG /* defined when there is no 64-bit support */
 
@@ -449,7 +449,7 @@ struct XXH64_state_s {
   XXH32_hash_t reserved32; /* required for padding anyway */
   XXH64_hash_t reserved64; /* never read nor write, might be removed in a future
                               version */
-}; /* typedef'd to XXH64_state_t */
+};                         /* typedef'd to XXH64_state_t */
 
 /*-**********************************************************************
  *  XXH3
@@ -954,9 +954,9 @@ static void* XXH_memcpy(void* dest, const void* src, size_t size) {
 /* *************************************
  *  Compiler Specific Options
  ***************************************/
-#ifdef _MSC_VER /* Visual Studio warning fix */
-#pragma warning( \
-    disable : 4127) /* disable: C4127: conditional expression is constant */
+#ifdef _MSC_VER                 /* Visual Studio warning fix */
+#pragma warning(disable : 4127) /* disable: C4127: conditional expression is \
+                                   constant */
 #endif
 
 #if XXH_NO_INLINE_HINTS /* disable inlining hints */
diff --git a/libCacheSim/dataStructure/minimalIncrementCBF.h b/libCacheSim/dataStructure/minimalIncrementCBF.h
@@ -1,5 +1,5 @@
-#ifndef _MINIMAL_INCREMENR_CBF_H
-#define _MINIMAL_INCREMENR_CBF_H
+#ifndef _MINIMAL_INCREMENT_CBF_H
+#define _MINIMAL_INCREMENT_CBF_H
 
 #ifdef __cplusplus
 extern "C" {
diff --git a/libCacheSim/dataStructure/robin_hood.h b/libCacheSim/dataStructure/robin_hood.h
@@ -615,27 +615,24 @@ struct pair {
 
   // pair constructors are explicit so we don't accidentally call this ctor when
   // we don't have to.
-  explicit constexpr pair(std::pair<T1, T2> const& o) noexcept(
-      noexcept(T1(std::declval<T1 const&>())) &&
-      noexcept(T2(std::declval<T2 const&>())))
+  explicit constexpr pair(std::pair<T1, T2> const& o) noexcept(noexcept(
+      T1(std::declval<T1 const&>())) && noexcept(T2(std::declval<T2 const&>())))
       : first(o.first), second(o.second) {}
 
   // pair constructors are explicit so we don't accidentally call this ctor when
   // we don't have to.
-  explicit constexpr pair(std::pair<T1, T2>&& o) noexcept(
-      noexcept(T1(std::move(std::declval<T1&&>()))) &&
-      noexcept(T2(std::move(std::declval<T2&&>()))))
+  explicit constexpr pair(std::pair<T1, T2>&& o) noexcept(noexcept(T1(std::move(
+      std::declval<T1&&>()))) && noexcept(T2(std::move(std::declval<T2&&>()))))
       : first(std::move(o.first)), second(std::move(o.second)) {}
 
-  constexpr pair(T1&& a, T2&& b) noexcept(
-      noexcept(T1(std::move(std::declval<T1&&>()))) &&
-      noexcept(T2(std::move(std::declval<T2&&>()))))
+  constexpr pair(T1&& a, T2&& b) noexcept(noexcept(T1(std::move(
+      std::declval<T1&&>()))) && noexcept(T2(std::move(std::declval<T2&&>()))))
       : first(std::move(a)), second(std::move(b)) {}
 
   template <typename U1, typename U2>
   constexpr pair(U1&& a, U2&& b) noexcept(
-      noexcept(T1(std::forward<U1>(std::declval<U1&&>()))) &&
-      noexcept(T2(std::forward<U2>(std::declval<U2&&>()))))
+      noexcept(T1(std::forward<U1>(std::declval<U1&&>()))) && noexcept(
+          T2(std::forward<U2>(std::declval<U2&&>()))))
       : first(std::forward<U1>(a)), second(std::forward<U2>(b)) {}
 
   template <typename... U1, typename... U2>
@@ -658,17 +655,13 @@ struct pair {
 
   // constructor called from the std::piecewise_construct_t ctor
   template <typename... U1, size_t... I1, typename... U2, size_t... I2>
-  pair(
-      std::tuple<U1...>& a, std::tuple<U2...>& b,
-      ROBIN_HOOD_STD::index_sequence<I1...> /*unused*/,
-      ROBIN_HOOD_STD::index_sequence<
-          I2...> /*unused*/) noexcept(noexcept(T1(std::
-                                                      forward<U1>(std::get<I1>(
-                                                          std::declval<std::tuple<
-                                                              U1...>&>()))...)) &&
-                                      noexcept(T2(std::forward<U2>(std::get<I2>(
-                                          std::declval<
-                                              std::tuple<U2...>&>()))...)))
+  pair(std::tuple<U1...>& a, std::tuple<U2...>& b, ROBIN_HOOD_STD::index_sequence<I1...> /*unused*/, ROBIN_HOOD_STD::index_sequence<I2...> /*unused*/) noexcept(
+      noexcept(T1(std::forward<U1>(std::get<I1>(
+          std::declval<std::tuple<
+              U1...>&>()))...)) && noexcept(T2(std::
+                                                   forward<U2>(std::get<I2>(
+                                                       std::declval<std::tuple<
+                                                           U2...>&>()))...)))
       : first(std::forward<U1>(std::get<I1>(a))...),
         second(std::forward<U2>(std::get<I2>(b))...) {
     // make visual studio compiler happy about warning about unused a & b.
@@ -705,8 +698,9 @@ inline constexpr bool operator!=(pair<A, B> const& x, pair<A, B> const& y) {
 template <typename A, typename B>
 inline constexpr bool
 operator<(pair<A, B> const& x, pair<A, B> const& y) noexcept(
-    noexcept(std::declval<A const&>() < std::declval<A const&>()) &&
-    noexcept(std::declval<B const&>() < std::declval<B const&>())) {
+    noexcept(std::declval<A const&>() <
+             std::declval<A const&>()) && noexcept(std::declval<B const&>() <
+                                                   std::declval<B const&>())) {
   return x.first < y.first || (!(y.first < x.first) && x.second < y.second);
 }
 template <typename A, typename B>
@@ -1552,8 +1546,8 @@ class Table
   // we can ignore it.
   explicit Table(
       size_t ROBIN_HOOD_UNUSED(bucket_count) /*unused*/, const Hash& h = Hash{},
-      const KeyEqual& equal = KeyEqual{}) noexcept(noexcept(Hash(h)) &&
-                                                   noexcept(KeyEqual(equal)))
+      const KeyEqual& equal =
+          KeyEqual{}) noexcept(noexcept(Hash(h)) && noexcept(KeyEqual(equal)))
       : WHash(h), WKeyEqual(equal) {
     ROBIN_HOOD_TRACE(this)
   }
diff --git a/libCacheSim/traceAnalyzer/reqRate.h b/libCacheSim/traceAnalyzer/reqRate.h
@@ -14,7 +14,7 @@ namespace traceAnalyzer {
 class ReqRate {
  public:
   ReqRate() = default;
-  explicit ReqRate(int time_window) : time_window_(time_window) {};
+  explicit ReqRate(int time_window) : time_window_(time_window){};
   ~ReqRate() = default;
 
   void add_req(request_t *req);
diff --git a/scripts/install_dev_dependency.sh b/scripts/install_dev_dependency.sh
@@ -46,7 +46,7 @@ main() {
 		sudo apt install -yqq nodejs npm
 	# trunk-ignore(shellcheck/SC2312)
 	elif [[ $(uname -a) == *"Darwin"* ]]; then
-		brew install gdb clang-format nodejs npm
+		brew install gdb clang-format flock nodejs npm
 	elif grep -qi 'microsoft' /proc/version 2>/dev/null; then
 		# WSL detection
 		sudo apt install -yqq gdb valgrind clang-tidy clang-format
diff --git a/scripts/setup_hooks.sh b/scripts/setup_hooks.sh
@@ -8,7 +8,7 @@ HOOKS_DIR="${REPO_ROOT}/.git/hooks"
 echo "Installing git hooks..."
 
 # Create pre-commit hook
-cat > "${HOOKS_DIR}/pre-commit" << 'EOL'
+cat >"${HOOKS_DIR}/pre-commit" <<'EOL'
 #!/bin/bash
 set -e
 
@@ -301,4 +301,4 @@ echo "Git hooks installed successfully!"
 echo "The pre-commit hook will now run automatically on each commit to check for linting issues."
 echo "You can bypass the checks with: SKIP_LINT=1 git commit"
 echo "Linting logs are stored in .lint-logs/ for future reference."
-echo "Note: clang-tidy and clang-format will be used if they're installed on your system." 
+echo "Note: clang-tidy and clang-format will be used if they're installed on your system."
