Merge pull request #1066 from dlyr/pre-commit-update

Pre commit update
STORM-IRIT · Oct 12, 2023 · e89ba19 · e89ba19
2 parents 629a228 + e937b79
commit e89ba19
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 10 deletions.
diff --git a/.clang-format b/.clang-format
@@ -1,11 +1,11 @@
 ---
 BasedOnStyle: LLVM
 Language: Cpp
-Standard: Cpp11
+Standard: c++17
 AllowShortFunctionsOnASingleLine: Inline
 AllowShortBlocksOnASingleLine: true
 AllowShortIfStatementsOnASingleLine: true
-AlwaysBreakTemplateDeclarations: true
+AlwaysBreakTemplateDeclarations: Yes
 BreakBeforeBraces: Custom
 BraceWrapping:
   AfterClass: true

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -5,7 +5,7 @@
 
 repos:
 - repo: https://github.com/pre-commit/pre-commit-hooks
-  rev: v4.4.0
+  rev: v4.5.0
   hooks:
     - id: trailing-whitespace
       exclude: ^tests/.*/data/.*$
@@ -26,12 +26,12 @@ repos:
 #      - id: cppcheck
 #        args: [--enable=all]
 - repo: https://github.com/igorshubovych/markdownlint-cli
-  rev: v0.33.0
+  rev: v0.37.0
   hooks:
   - id: markdownlint
     args: [-f]
 - repo: https://github.com/pre-commit/mirrors-clang-format
-  rev: v15.0.6
+  rev: v17.0.2
   hooks:
   - id: clang-format
     types_or: [c++, c, cuda]
diff --git a/src/Core/Containers/VariableSet.hpp b/src/Core/Containers/VariableSet.hpp
@@ -719,7 +719,7 @@ auto VariableSet::addVariableType() -> Utils::optional<VariableContainer<T>*> {
         // used to visit the variableSet with a dynamic visitor
         m_vtable->m_visitFunctions.emplace_back(
             []( const VariableSet& c, const DynamicVisitorBase& v )
-                -> std::pair<bool, std::function<void( DynamicVisitorBase&, std::any && )>> {
+                -> std::pair<bool, std::function<void( DynamicVisitorBase&, std::any&& )>> {
                 auto id = getVariableVisitTypeIndex<T>();
                 if ( v.accept( id ) ) {
                     auto& visitedStorage = c.getVariableStorage<T>();

diff --git a/src/Core/Utils/Attribs.hpp b/src/Core/Utils/Attribs.hpp
@@ -453,7 +453,7 @@ void AttribBase::setName( const std::string& name ) {
     m_name = name;
 }
 
-bool inline AttribBase::operator==( const AttribBase& rhs ) {
+bool inline AttribBase::operator==( const AttribBase & rhs ) {
     return m_name == rhs.getName();
 }
 

diff --git a/src/Core/Utils/Singleton.hpp b/src/Core/Utils/Singleton.hpp
@@ -23,7 +23,9 @@
     TYPE( const TYPE& )           = delete;                                            \
     void operator=( const TYPE& ) = delete;                                            \
     struct Deleter {                                                                   \
-        void operator()( TYPE* p ) const { delete p; }                                 \
+        void operator()( TYPE* p ) const {                                             \
+            delete p;                                                                  \
+        }                                                                              \
     };                                                                                 \
     static std::unique_ptr<TYPE, Deleter> s_instance;                                  \
                                                                                        \
@@ -33,8 +35,12 @@
         s_instance = std::unique_ptr<TYPE, Deleter>( new TYPE( args... ), Deleter() ); \
         return getInstance();                                                          \
     }                                                                                  \
-    inline static TYPE* getInstance() { return s_instance.get(); }                     \
-    inline static void destroyInstance() { s_instance.reset( nullptr ); }
+    inline static TYPE* getInstance() {                                                \
+        return s_instance.get();                                                       \
+    }                                                                                  \
+    inline static void destroyInstance() {                                             \
+        s_instance.reset( nullptr );                                                   \
+    }
 
 /// Add this macro in the singleton cpp, followed by a semicolon.
 // Limitations : TYPE cannot be a nested type