chore: update pre-commit hooks (#1137)

* chore: update pre-commit hooks

updates:
- [github.com/pre-commit/mirrors-clang-format: v18.1.8 → v19.1.0](pre-commit/mirrors-clang-format@v18.1.8...v19.1.0)

* style: pre-commit fixes

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

.pre-commit-config.yaml

@@ -65,7 +65,7 @@ repos:
 
 # clang-format
 - repo: https://github.com/pre-commit/mirrors-clang-format
-  rev: v18.1.8
+  rev: v19.1.0
   hooks:
   - id: clang-format
     exclude: ^external/.*

include/slang/util/TimeTrace.h

@@ -56,22 +56,21 @@ class SLANG_EXPORT TimeTrace {
 /// When the object is constructed, it begins the trace section and when
 /// it is destroyed, it stops it. If the time profiler is not initialized
 /// the overhead is a single branch.
-class SLANG_EXPORT TimeTraceScope {
-public:
-    TimeTraceScope(std::string_view name, std::string_view detail) {
-        if (TimeTrace::isEnabled())
-            TimeTrace::beginTrace(name, detail);
-    }
+class SLANG_EXPORT
+TimeTraceScope{public: TimeTraceScope(std::string_view name, std::string_view detail){
+    if (TimeTrace::isEnabled()) TimeTrace::beginTrace(name, detail);
+} // namespace slang
 
-    TimeTraceScope(std::string_view name, function_ref<std::string()> detail) {
-        if (TimeTrace::isEnabled())
-            TimeTrace::beginTrace(name, detail);
-    }
+TimeTraceScope(std::string_view name, function_ref<std::string()> detail) {
+    if (TimeTrace::isEnabled())
+        TimeTrace::beginTrace(name, detail);
+}
 
-    ~TimeTraceScope() {
-        if (TimeTrace::isEnabled())
-            TimeTrace::endTrace();
-    }
-};
+~TimeTraceScope() {
+    if (TimeTrace::isEnabled())
+        TimeTrace::endTrace();
+}
+}
+;
 
 } // namespace slang

source/ast/Compilation.cpp

@@ -2392,8 +2392,8 @@ void Compilation::resolveDefParamsAndBinds() {
 }
 
 template<typename TDefList>
-auto findDefByLib(TDefList& defList,
-                  const SourceLibrary& target) -> std::remove_reference_t<decltype(defList[0])> {
+auto findDefByLib(TDefList& defList, const SourceLibrary& target)
+    -> std::remove_reference_t<decltype(defList[0])> {
     for (auto def : defList) {
         if (def->getSourceLibrary() == &target)
             return def;

tests/unittests/util/NumericTests.cpp

@@ -230,7 +230,9 @@ TEST_CASE("SVInt to string (and back)") {
     CHECK(ss.str() == "1");
 
     // This caught a heap corruption bug in from-string truncation.
-    { "54'bxx0111x10xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"_si; }
+    {
+        "54'bxx0111x10xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"_si;
+    }
 
     CHECK("96'hx00000Z101"_si.toString(LiteralBase::Decimal, false) == "X");
     CHECK("96'hxxxxxxxxxxxxxxxxxxxxxxxx"_si.toString(LiteralBase::Decimal, false) == "x");