From 1257ac21bf454a04cf5f5ef4f880650499e4caa9 Mon Sep 17 00:00:00 2001
From: "pre-commit-ci[bot]"
 <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Date: Mon, 7 Oct 2024 16:30:23 -0700
Subject: [PATCH] [pre-commit.ci] pre-commit autoupdate (#1631)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.6.0 → v5.0.0](https://github.com/pre-commit/pre-commit-hooks/compare/v4.6.0...v5.0.0)
- [github.com/pre-commit/mirrors-clang-format: v18.1.5 → v19.1.1](https://github.com/pre-commit/mirrors-clang-format/compare/v18.1.5...v19.1.1)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
---
 .pre-commit-config.yaml                  |  4 ++--
 include/openPMD/RecordComponent.hpp      |  3 +--
 include/openPMD/backend/Attributable.hpp | 12 +++++-------
 src/IO/AbstractIOHandlerImpl.cpp         |  4 ++--
 src/backend/Attributable.cpp             | 12 +++++-------
 5 files changed, 15 insertions(+), 20 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 61147bc862..ca7aacb5b1 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -18,7 +18,7 @@ exclude: '^share/openPMD/thirdParty'
 # See https://pre-commit.com/hooks.html for more hooks
 repos:
 -   repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.6.0
+    rev: v5.0.0
     hooks:
     - id: trailing-whitespace
       args: [--markdown-linebreak-ext=md]
@@ -66,7 +66,7 @@ repos:
 # clang-format v13
 #   to run manually, use .github/workflows/clang-format/clang-format.sh
 - repo: https://github.com/pre-commit/mirrors-clang-format
-  rev: v18.1.5
+  rev: v19.1.1
   hooks:
   - id: clang-format
     # By default, the clang-format hook configures:
diff --git a/include/openPMD/RecordComponent.hpp b/include/openPMD/RecordComponent.hpp
index ebb5a80ca8..9072b93a32 100644
--- a/include/openPMD/RecordComponent.hpp
+++ b/include/openPMD/RecordComponent.hpp
@@ -482,8 +482,7 @@ class RecordComponent : public BaseRecordComponent
      */
     template <typename Visitor, typename... Args>
     auto visit(Args &&...args) -> decltype(Visitor::template call<char>(
-                                   std::declval<RecordComponent &>(),
-                                   std::forward<Args>(args)...));
+        std::declval<RecordComponent &>(), std::forward<Args>(args)...));
 
     static constexpr char const *const SCALAR = "\vScalar";
 
diff --git a/include/openPMD/backend/Attributable.hpp b/include/openPMD/backend/Attributable.hpp
index 08090ce2e3..850d3bf35c 100644
--- a/include/openPMD/backend/Attributable.hpp
+++ b/include/openPMD/backend/Attributable.hpp
@@ -341,13 +341,11 @@ OPENPMD_protected
      * Throws an error otherwise, e.g., for Series objects.
      * @{
      */
-    [[nodiscard]] auto containingIteration() const
-        -> std::pair<
-            std::optional<internal::IterationData const *>,
-            internal::SeriesData const *>;
-    auto containingIteration() -> std::pair<
-                                   std::optional<internal::IterationData *>,
-                                   internal::SeriesData *>;
+    [[nodiscard]] auto containingIteration() const -> std::pair<
+        std::optional<internal::IterationData const *>,
+        internal::SeriesData const *>;
+    auto containingIteration() -> std::
+        pair<std::optional<internal::IterationData *>, internal::SeriesData *>;
     /** @} */
 
     template <bool flush_entire_series>
diff --git a/src/IO/AbstractIOHandlerImpl.cpp b/src/IO/AbstractIOHandlerImpl.cpp
index 8993816f48..7675cc3e07 100644
--- a/src/IO/AbstractIOHandlerImpl.cpp
+++ b/src/IO/AbstractIOHandlerImpl.cpp
@@ -81,8 +81,8 @@ namespace
     using self_or_invoked_t = typename self_or_invoked<T>::type;
 
     template <typename DeferredString>
-    auto
-    undefer_string(DeferredString &&str) -> self_or_invoked_t<DeferredString &&>
+    auto undefer_string(DeferredString &&str)
+        -> self_or_invoked_t<DeferredString &&>
     {
         if constexpr (std::is_invocable_v<DeferredString &&>)
         {
diff --git a/src/backend/Attributable.cpp b/src/backend/Attributable.cpp
index 018a776a58..9ffbe25ee3 100644
--- a/src/backend/Attributable.cpp
+++ b/src/backend/Attributable.cpp
@@ -140,10 +140,9 @@ Series Attributable::retrieveSeries() const
     return findSeries->attributable->asInternalCopyOf<Series>();
 }
 
-auto Attributable::containingIteration() const
-    -> std::pair<
-        std::optional<internal::IterationData const *>,
-        internal::SeriesData const *>
+auto Attributable::containingIteration() const -> std::pair<
+    std::optional<internal::IterationData const *>,
+    internal::SeriesData const *>
 {
     constexpr size_t search_queue_size = 3;
     Writable const *search_queue[search_queue_size]{nullptr};
@@ -182,9 +181,8 @@ auto Attributable::containingIteration() const
     }
 }
 
-auto Attributable::containingIteration()
-    -> std::
-        pair<std::optional<internal::IterationData *>, internal::SeriesData *>
+auto Attributable::containingIteration() -> std::
+    pair<std::optional<internal::IterationData *>, internal::SeriesData *>
 {
     auto const_res =
         static_cast<Attributable const *>(this)->containingIteration();