commit 378903928085318c204952400754d4a82b47f1e5
author Patrick Venture <venture@google.com> Wed Sep 25 17:05:03 2019 -0700
committer Patrick Venture <venture@google.com> Thu Sep 26 16:37:35 2019 +0000
tree 29ecdd499f20c9923780be4542c3029ec7f503b9
parent a41714c5a15af97d5b3996c77c8ad2d379fd7af9

manufacturingcommands: pass string by const ref

[src/manufacturingcommands.cpp:89]:
(performance) Function parameter 'setState' should be passed by const
reference.

Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Ie22127f4d071d684d646d047178e639826537437

src/manufacturingcommands.cpp

diff --git a/src/manufacturingcommands.cpp b/src/manufacturingcommands.cpp
index 5e3dbdd..31e8938 100644
--- a/src/manufacturingcommands.cpp
+++ b/src/manufacturingcommands.cpp
@@ -86,7 +86,7 @@
     return 0;
 }
 
-ipmi_ret_t ledStoreAndSet(SmSignalSet signal, std::string setState)
+ipmi_ret_t ledStoreAndSet(SmSignalSet signal, const std::string& setState)
 {
     LedProperty* ledProp = mtm.findLedProperty(signal);
     if (ledProp == nullptr)