Add further cleanup

ecmwf · Nov 21, 2024 · e33bd0d · e33bd0d
1 parent 03cda38
commit e33bd0d
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/src/multio/action/scale/Scale.cc b/src/multio/action/scale/Scale.cc
@@ -34,15 +34,14 @@ Scale::Scale(const ComponentConfiguration& compConf) :
     const auto mappings = compConf.parsedConfig().has("mapping-definition")
                             ? compConf.parsedConfig().getSubConfigurations("mapping-definition")
                             : throw eckit::SeriousBug{"Scaling information not specified in plan", Here()};
-    ;
 
     if (!mappings.empty()) {
         for (const auto& mapping : mappings) {
             auto matcher = mapping.getSubConfiguration("case");
             paramsToScale_.insert(matcher.getString("param-is"));
         }
     }
-    std::cout << paramsToScale_ << std::endl;
+    eckit::Log::info() << paramsToScale_ << std::endl;
 }
 
 void Scale::executeImpl(message::Message msg) {
@@ -102,4 +101,4 @@ template message::Message Scale::ScaleMessage<float>(message::Message&&) const;
 template message::Message Scale::ScaleMessage<double>(message::Message&&) const;
 
 
-}  // namespace multio::action
+}  // namespace multio::action
diff --git a/src/multio/action/scale/Scale.h b/src/multio/action/scale/Scale.h
@@ -11,7 +11,8 @@ namespace multio::action {
 class Scale final : public ChainedAction {
 public:
     explicit Scale(const ComponentConfiguration& compConf);  // Constructor declaration
-    void executeImpl(message::Message) override;
+
+    void executeImpl(message::Message msg) override;
 
 private:
     ScaleScaling scaling_;

diff --git a/src/multio/action/scale/Scaling.h b/src/multio/action/scale/Scaling.h
@@ -20,6 +20,7 @@ class ScaleScaling {
 public:
     explicit ScaleScaling(const config::ComponentConfiguration& compConf);
     double getScalingFactor(const std::string paramID) const;
+
     template <typename Precision>
     void applyScaling(message::Message& msg) const {
         if (hasScaling_) {
@@ -66,4 +67,4 @@ class ScaleScaling {
     }
 };
 
-}  // namespace multio::action
+}  // namespace multio::action