clang-tidy: Enable readability-const-return-type check
Checks for functions with a const-qualified return type and
recommends removal of the const keyword. Such use of const is usually
superfluous, and can prevent valuable compiler optimizations. Does
not (yet) fix trailing return types.
Signed-off-by: George Liu <liuxiwei@ieisystem.com>
Change-Id: Ifff7541c95d7881d8c6407b20c906ec7eb13abf1
diff --git a/fault-monitor/operational-status-monitor.cpp b/fault-monitor/operational-status-monitor.cpp
index e5b0c7f..10299f7 100644
--- a/fault-monitor/operational-status-monitor.cpp
+++ b/fault-monitor/operational-status-monitor.cpp
@@ -55,7 +55,7 @@
}
}
-const std::vector<std::string>
+std::vector<std::string>
Monitor::getLedGroupPaths(const std::string& inventoryPath) const
{
// Get endpoints from fType
diff --git a/fault-monitor/operational-status-monitor.hpp b/fault-monitor/operational-status-monitor.hpp
index 6bd2efc..751b970 100644
--- a/fault-monitor/operational-status-monitor.hpp
+++ b/fault-monitor/operational-status-monitor.hpp
@@ -79,7 +79,7 @@
*
* @return std::vector<std::string> - Vector of LED Group D-Bus object paths
*/
- const std::vector<std::string>
+ std::vector<std::string>
getLedGroupPaths(const std::string& inventoryPath) const;
/**