From dd088f952e001d6e7e49511c669eb9e7fd770da0 Mon Sep 17 00:00:00 2001 From: "pre-commit-ci[bot]" <66853113+pre-commit-ci[bot]@users.noreply.github.com> Date: Mon, 3 Feb 2025 21:49:45 +0000 Subject: [PATCH 1/2] [pre-commit.ci] pre-commit autoupdate MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit updates: - [github.com/pre-commit/mirrors-clang-format: v17.0.3 → v19.1.7](https://github.com/pre-commit/mirrors-clang-format/compare/v17.0.3...v19.1.7) - [github.com/pre-commit/pre-commit-hooks: v4.5.0 → v5.0.0](https://github.com/pre-commit/pre-commit-hooks/compare/v4.5.0...v5.0.0) - [github.com/psf/black: 23.10.0 → 25.1.0](https://github.com/psf/black/compare/23.10.0...25.1.0) - [github.com/PyCQA/flake8: 6.1.0 → 7.1.1](https://github.com/PyCQA/flake8/compare/6.1.0...7.1.1) --- .pre-commit-config.yaml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 51472bd8..aeaf4a3c 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -2,12 +2,12 @@ ci: autoupdate_branch: 'devel' repos: - repo: https://github.com/pre-commit/mirrors-clang-format - rev: v17.0.3 + rev: v19.1.7 hooks: - id: clang-format args: ['--style={BasedOnStyle: Google,SortIncludes: false}'] - repo: https://github.com/pre-commit/pre-commit-hooks - rev: v4.5.0 + rev: v5.0.0 hooks: - id: check-added-large-files - id: check-ast @@ -25,11 +25,11 @@ repos: - id: mixed-line-ending - id: trailing-whitespace - repo: https://github.com/psf/black - rev: 23.10.0 + rev: 25.1.0 hooks: - id: black - repo: https://github.com/PyCQA/flake8 - rev: 6.1.0 + rev: 7.1.1 hooks: - id: flake8 - repo: https://github.com/cheshirekow/cmake-format-precommit From 2a729d3f925a8520f9e9ec9c3ea83ece16d844f4 Mon Sep 17 00:00:00 2001 From: "pre-commit-ci[bot]" <66853113+pre-commit-ci[bot]@users.noreply.github.com> Date: Mon, 3 Feb 2025 21:50:02 +0000 Subject: [PATCH 2/2] [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --- include/sot/core/binary-op.hh | 2 +- include/sot/core/derivator.hh | 2 +- include/sot/core/feature-abstract.hh | 2 +- include/sot/core/latch.hh | 2 +- include/sot/core/parameter-server.hh | 2 +- include/sot/core/sequencer.hh | 2 +- include/sot/core/smooth-reach.hh | 2 +- include/sot/core/sot.hh | 3 +-- include/sot/core/task-abstract.hh | 4 ++-- include/sot/core/unary-op.hh | 2 +- include/sot/core/variadic-op.hh | 2 +- src/filters/filter-differentiator.cpp | 14 +++++++------- tests/tools/dummy-sot-external-interface.hh | 4 ++-- tests/tools/plugin.cc | 4 ++-- tests/tools/plugin.hh | 4 ++-- tests/tools/test_abstract_interface.cpp | 4 ++-- 16 files changed, 27 insertions(+), 28 deletions(-) diff --git a/include/sot/core/binary-op.hh b/include/sot/core/binary-op.hh index 5bdc462a..af15b543 100644 --- a/include/sot/core/binary-op.hh +++ b/include/sot/core/binary-op.hh @@ -65,7 +65,7 @@ class BinaryOp : public Entity { op.addSpecificCommands(*this, commandMap); } - virtual ~BinaryOp(void){}; + virtual ~BinaryOp(void) {}; public: /* --- SIGNAL --- */ SignalPtr SIN1; diff --git a/include/sot/core/derivator.hh b/include/sot/core/derivator.hh index 2829503a..dd716744 100644 --- a/include/sot/core/derivator.hh +++ b/include/sot/core/derivator.hh @@ -65,7 +65,7 @@ class Derivator : public dynamicgraph::Entity { timestepSIN.setKeepReference(true); } - virtual ~Derivator(void){}; + virtual ~Derivator(void) {}; public: /* --- SIGNAL --- */ dynamicgraph::SignalPtr SIN; diff --git a/include/sot/core/feature-abstract.hh b/include/sot/core/feature-abstract.hh index ff361d2e..c503e9c0 100644 --- a/include/sot/core/feature-abstract.hh +++ b/include/sot/core/feature-abstract.hh @@ -90,7 +90,7 @@ class SOT_CORE_EXPORT FeatureAbstract : public Entity { /*! \brief Default constructor: the name of the class should be given. */ FeatureAbstract(const std::string &name); /*! \brief Default destructor */ - virtual ~FeatureAbstract(void){}; + virtual ~FeatureAbstract(void) {}; /*! \name Methods returning the dimension of the feature. @{ */ diff --git a/include/sot/core/latch.hh b/include/sot/core/latch.hh index e2e516ff..3e42d307 100644 --- a/include/sot/core/latch.hh +++ b/include/sot/core/latch.hh @@ -75,7 +75,7 @@ class Latch : public Entity { docCommandVoid0("Turn off the latch"))); } - virtual ~Latch(void){}; + virtual ~Latch(void) {}; }; } /* namespace sot */ } /* namespace dynamicgraph */ diff --git a/include/sot/core/parameter-server.hh b/include/sot/core/parameter-server.hh index 5f1cd449..158fc241 100644 --- a/include/sot/core/parameter-server.hh +++ b/include/sot/core/parameter-server.hh @@ -62,7 +62,7 @@ class SOTParameterServer_EXPORT ParameterServer /* --- CONSTRUCTOR ---- */ ParameterServer(const std::string &name); - ~ParameterServer(){}; + ~ParameterServer() {}; /// Initialize /// @param dt: control interval diff --git a/include/sot/core/sequencer.hh b/include/sot/core/sequencer.hh index 2a063c8c..75692cbb 100644 --- a/include/sot/core/sequencer.hh +++ b/include/sot/core/sequencer.hh @@ -65,7 +65,7 @@ class SOTSEQUENCER_EXPORT Sequencer : public dynamicgraph::Entity { size_type eventType; public: - sotEventAbstract(const std::string &name) : name(name){}; + sotEventAbstract(const std::string &name) : name(name) {}; virtual ~sotEventAbstract(void) {} virtual const std::string &getName() const { return name; } size_type getEventType() const { return eventType; } diff --git a/include/sot/core/smooth-reach.hh b/include/sot/core/smooth-reach.hh index e3806b55..46a608c1 100644 --- a/include/sot/core/smooth-reach.hh +++ b/include/sot/core/smooth-reach.hh @@ -58,7 +58,7 @@ class SOTSMOOTHREACH_EXPORT SmoothReach : public dynamicgraph::Entity { public: /* --- CONSTRUCTION --- */ SmoothReach(const std::string &name); - virtual ~SmoothReach(void){}; + virtual ~SmoothReach(void) {}; public: /* --- SIGNAL --- */ dynamicgraph::SignalPtr startSIN; diff --git a/include/sot/core/sot.hh b/include/sot/core/sot.hh index 34ebbeb6..5efa6ad0 100644 --- a/include/sot/core/sot.hh +++ b/include/sot/core/sot.hh @@ -101,8 +101,7 @@ class SOTSOT_CORE_EXPORT Sot : public Entity { public: /*! \brief Default constructor */ Sot(const std::string &name); - ~Sot(void) { /* TODO!! */ - } + ~Sot(void) { /* TODO!! */ } /*! \name Methods to handle the stack. @{ diff --git a/include/sot/core/task-abstract.hh b/include/sot/core/task-abstract.hh index b1c54177..d6dd186e 100644 --- a/include/sot/core/task-abstract.hh +++ b/include/sot/core/task-abstract.hh @@ -53,8 +53,8 @@ class SOT_CORE_EXPORT TaskAbstract : public dynamicgraph::Entity { /* Use a derivative of this class to store computational memory. */ class MemoryTaskAbstract { public: - MemoryTaskAbstract(void){}; - virtual ~MemoryTaskAbstract(void){}; + MemoryTaskAbstract(void) {}; + virtual ~MemoryTaskAbstract(void) {}; public: virtual void display(std::ostream &os) const = 0; diff --git a/include/sot/core/unary-op.hh b/include/sot/core/unary-op.hh index 6e29e854..98b09b59 100644 --- a/include/sot/core/unary-op.hh +++ b/include/sot/core/unary-op.hh @@ -52,7 +52,7 @@ class UnaryOp : public Entity { op.addSpecificCommands(*this, commandMap); } - virtual ~UnaryOp(void){}; + virtual ~UnaryOp(void) {}; public: /* --- SIGNAL --- */ SignalPtr SIN; diff --git a/include/sot/core/variadic-op.hh b/include/sot/core/variadic-op.hh index 9557f0df..98d412f5 100644 --- a/include/sot/core/variadic-op.hh +++ b/include/sot/core/variadic-op.hh @@ -159,7 +159,7 @@ class VariadicOp : public VariadicAbstractcommandMap); } - virtual ~VariadicOp(void){}; + virtual ~VariadicOp(void) {}; protected: Tout &computeOperation(Tout &res, sigtime_t time) { diff --git a/src/filters/filter-differentiator.cpp b/src/filters/filter-differentiator.cpp index 687d5735..a139817f 100644 --- a/src/filters/filter-differentiator.cpp +++ b/src/filters/filter-differentiator.cpp @@ -97,19 +97,19 @@ void FilterDifferentiator::init(const double ×tep, const size_type &xSize, m_filter = new CausalFilter(timestep, xSize, filter_numerator, filter_denominator); - LOG("Filtering started with " - << "Numerator " << filter_numerator << std::endl - << "Denominator" << filter_denominator << std::endl); + LOG("Filtering started with " << "Numerator " << filter_numerator << std::endl + << "Denominator" << filter_denominator + << std::endl); return; } void FilterDifferentiator::switch_filter( const Eigen::VectorXd &filter_numerator, const Eigen::VectorXd &filter_denominator) { - LOG("Filter switched with " - << "Numerator " << filter_numerator << std::endl - << "Denominator" << filter_denominator << std::endl - << "at time" << m_xSIN.getTime()); + LOG("Filter switched with " << "Numerator " << filter_numerator << std::endl + << "Denominator" << filter_denominator + << std::endl + << "at time" << m_xSIN.getTime()); m_filter->switch_filter(filter_numerator, filter_denominator); } diff --git a/tests/tools/dummy-sot-external-interface.hh b/tests/tools/dummy-sot-external-interface.hh index 4882d724..d94165b8 100644 --- a/tests/tools/dummy-sot-external-interface.hh +++ b/tests/tools/dummy-sot-external-interface.hh @@ -19,9 +19,9 @@ typedef dynamicgraph::size_type size_type; class DummySotExternalInterface : public dynamicgraph::sot::AbstractSotExternalInterface { public: - DummySotExternalInterface(){}; + DummySotExternalInterface() {}; - virtual ~DummySotExternalInterface(){}; + virtual ~DummySotExternalInterface() {}; virtual void setupSetSensors( std::map &sensorsIn); diff --git a/tests/tools/plugin.cc b/tests/tools/plugin.cc index fc20ef52..20664d78 100644 --- a/tests/tools/plugin.cc +++ b/tests/tools/plugin.cc @@ -27,8 +27,8 @@ class Plugin : public PluginAbstract { AbstractSotExternalInterface *sotController_; public: - Plugin(){}; - ~Plugin(){}; + Plugin() {}; + ~Plugin() {}; void Initialization(std::string &dynamicLibraryName) { // Load the SotRobotBipedController library. diff --git a/tests/tools/plugin.hh b/tests/tools/plugin.hh index ce5bb4e3..1325aee9 100644 --- a/tests/tools/plugin.hh +++ b/tests/tools/plugin.hh @@ -3,8 +3,8 @@ class PluginAbstract { public: - PluginAbstract(){}; - virtual ~PluginAbstract(){}; + PluginAbstract() {}; + virtual ~PluginAbstract() {}; virtual void Initialization(std::string &astr) = 0; }; diff --git a/tests/tools/test_abstract_interface.cpp b/tests/tools/test_abstract_interface.cpp index c9337eb1..520263ba 100644 --- a/tests/tools/test_abstract_interface.cpp +++ b/tests/tools/test_abstract_interface.cpp @@ -30,8 +30,8 @@ class PluginLoader { std::string dynamicLibraryName_; public: - PluginLoader(){}; - ~PluginLoader(){}; + PluginLoader() {}; + ~PluginLoader() {}; int parseOptions(int argc, char *argv[]) { po::options_description desc("Allowed options");