From 417cd69fca23f29a4eebcfd33e4b62b205b077da Mon Sep 17 00:00:00 2001 From: Samuel Gaist Date: Thu, 11 Sep 2025 09:36:04 +0200 Subject: [PATCH] chore: pre-commit cleanup Signed-off-by: Samuel Gaist --- unit_tests/test_falco_engine.cpp | 4 +--- userspace/engine/yaml_helper.h | 29 ++++++++++++++--------------- userspace/falco/outputs.h | 2 +- userspace/falco/outputs_http.cpp | 5 ++++- 4 files changed, 20 insertions(+), 20 deletions(-) diff --git a/unit_tests/test_falco_engine.cpp b/unit_tests/test_falco_engine.cpp index 90f3a395..8ed1cf16 100644 --- a/unit_tests/test_falco_engine.cpp +++ b/unit_tests/test_falco_engine.cpp @@ -1,8 +1,6 @@ #include "test_falco_engine.h" -test_falco_engine::test_falco_engine(): - m_engine(std::make_shared()) -{ +test_falco_engine::test_falco_engine(): m_engine(std::make_shared()) { // create a falco engine ready to load the ruleset m_filter_factory = std::make_shared(&m_inspector, m_filterlist); m_formatter_factory = std::make_shared(&m_inspector, m_filterlist); diff --git a/userspace/engine/yaml_helper.h b/userspace/engine/yaml_helper.h index dcb9c66b..667e8be5 100644 --- a/userspace/engine/yaml_helper.h +++ b/userspace/engine/yaml_helper.h @@ -471,22 +471,21 @@ struct convert { res.emplace_back(sub); } break; - case YAML::NodeType::Scalar: - { - int int_val; - double double_val; - bool bool_val; - std::string str_val; - if(YAML::convert::decode(node, int_val)) { - res = int_val; - } else if(YAML::convert::decode(node, double_val)) { - res = double_val; - } else if(YAML::convert::decode(node, bool_val)) { - res = bool_val; - } else if(YAML::convert::decode(node, str_val)) { - res = str_val; - } + case YAML::NodeType::Scalar: { + int int_val; + double double_val; + bool bool_val; + std::string str_val; + if(YAML::convert::decode(node, int_val)) { + res = int_val; + } else if(YAML::convert::decode(node, double_val)) { + res = double_val; + } else if(YAML::convert::decode(node, bool_val)) { + res = bool_val; + } else if(YAML::convert::decode(node, str_val)) { + res = str_val; } + } default: break; } diff --git a/userspace/falco/outputs.h b/userspace/falco/outputs.h index 94e63cb7..394639cc 100644 --- a/userspace/falco/outputs.h +++ b/userspace/falco/outputs.h @@ -74,7 +74,7 @@ public: } // Return the output's name as per its configuration. - const std::string &get_name() const { return m_oc.name; } + const std::string& get_name() const { return m_oc.name; } // Output a message. virtual void output(const message* msg) = 0; diff --git a/userspace/falco/outputs_http.cpp b/userspace/falco/outputs_http.cpp index e2478955..9ef60d7b 100644 --- a/userspace/falco/outputs_http.cpp +++ b/userspace/falco/outputs_http.cpp @@ -20,7 +20,10 @@ limitations under the License. #define CHECK_RES(fn) res = res == CURLE_OK ? fn : res -static size_t noop_write_callback(void */*contents*/, size_t size, size_t nmemb, void */*userp*/) { +static size_t noop_write_callback(void * /*contents*/, + size_t size, + size_t nmemb, + void * /*userp*/) { // We don't want to echo anything. Just return size of bytes ignored return size * nmemb; }