[nifi-minifi-cpp] 05/05: Review update

[lordgamez]

This is an automated email from the ASF dual-hosted git repository.

lordgamez pushed a commit to branch MINIFICPP-2152
in repository https://gitbox.apache.org/repos/asf/nifi-minifi-cpp.git

commit 85ec9ef8d8cbc4e0722709bf1ce06c269156be28
Author: Gabor Gyimesi <gamezb...@gmail.com>
AuthorDate: Tue Aug 8 13:48:29 2023 +0200

    Review update
---
 libminifi/include/core/Processor.h | 5 +++--
 libminifi/src/core/Processor.cpp   | 8 ++++----
 libminifi/test/DummyProcessor.h    | 6 +++---
 3 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/libminifi/include/core/Processor.h 
b/libminifi/include/core/Processor.h
index d8eb5632d..477b6b29a 100644
--- a/libminifi/include/core/Processor.h
+++ b/libminifi/include/core/Processor.h
@@ -26,6 +26,7 @@
 #include <memory>
 #include <mutex>
 #include <string>
+#include <string_view>
 #include <unordered_set>
 #include <unordered_map>
 #include <utility>
@@ -73,8 +74,8 @@ constexpr std::chrono::microseconds 
MINIMUM_SCHEDULING_PERIOD{30};
 
 class Processor : public Connectable, public ConfigurableComponent, public 
state::response::ResponseNodeSource {
  public:
-  Processor(std::string name, const utils::Identifier& uuid, 
std::shared_ptr<ProcessorMetrics> metrics = nullptr);
-  explicit Processor(std::string name, std::shared_ptr<ProcessorMetrics> 
metrics = nullptr);
+  Processor(std::string_view name, const utils::Identifier& uuid, 
std::shared_ptr<ProcessorMetrics> metrics = nullptr);
+  explicit Processor(std::string_view name, std::shared_ptr<ProcessorMetrics> 
metrics = nullptr);
 
   Processor(const Processor& parent) = delete;
   Processor& operator=(const Processor& parent) = delete;
diff --git a/libminifi/src/core/Processor.cpp b/libminifi/src/core/Processor.cpp
index cf2449bc1..c865442cc 100644
--- a/libminifi/src/core/Processor.cpp
+++ b/libminifi/src/core/Processor.cpp
@@ -40,8 +40,8 @@ using namespace std::literals::chrono_literals;
 
 namespace org::apache::nifi::minifi::core {
 
-Processor::Processor(std::string name, std::shared_ptr<ProcessorMetrics> 
metrics)
-    : Connectable(std::move(name)),
+Processor::Processor(std::string_view name, std::shared_ptr<ProcessorMetrics> 
metrics)
+    : Connectable(name),
       logger_(logging::LoggerFactory<Processor>::getLogger(uuid_)),
       metrics_(metrics ? std::move(metrics) : 
std::make_shared<ProcessorMetrics>(*this)) {
   has_work_.store(false);
@@ -59,8 +59,8 @@ Processor::Processor(std::string name, 
std::shared_ptr<ProcessorMetrics> metrics
   logger_->log_debug("Processor %s created UUID %s", name_, getUUIDStr());
 }
 
-Processor::Processor(std::string name, const utils::Identifier& uuid, 
std::shared_ptr<ProcessorMetrics> metrics)
-    : Connectable(std::move(name), uuid),
+Processor::Processor(std::string_view name, const utils::Identifier& uuid, 
std::shared_ptr<ProcessorMetrics> metrics)
+    : Connectable(name, uuid),
       logger_(logging::LoggerFactory<Processor>::getLogger(uuid_)),
       metrics_(metrics ? std::move(metrics) : 
std::make_shared<ProcessorMetrics>(*this)) {
   has_work_.store(false);
diff --git a/libminifi/test/DummyProcessor.h b/libminifi/test/DummyProcessor.h
index 9a88d55d6..daa8092b2 100644
--- a/libminifi/test/DummyProcessor.h
+++ b/libminifi/test/DummyProcessor.h
@@ -17,7 +17,7 @@
 #pragma once
 
 #include <array>
-#include <string>
+#include <string_view>
 #include <utility>
 
 #include "core/Processor.h"
@@ -29,8 +29,8 @@ class DummyProcessor : public minifi::core::Processor {
   using minifi::core::Processor::Processor;
 
  public:
-  DummyProcessor(std::string name, const minifi::utils::Identifier& uuid) : 
Processor(std::move(name), uuid) {}
-  explicit DummyProcessor(std::string name) : Processor(std::move(name)) {}
+  DummyProcessor(std::string_view name, const minifi::utils::Identifier& uuid) 
: Processor(std::move(name), uuid) {}
+  explicit DummyProcessor(std::string_view name) : Processor(std::move(name)) 
{}
   static constexpr const char* Description = "A processor that does nothing.";
   static constexpr auto Properties = std::array<core::PropertyReference, 0>{};
   static constexpr auto Relationships = 
std::array<core::RelationshipDefinition, 0>{};