Clang-tidy fixes

Author: urfeex

include/ur_client_library/comm/pipeline.h

@@ -112,7 +112,7 @@ class MultiConsumer : public IConsumer<T>
    */
   void addConsumer(std::shared_ptr<IConsumer<T>> consumer)
   {
-    std::lock_guard<std::mutex> lk(consumer_list);
+    std::lock_guard<std::mutex> lk(consumer_list_);
     consumers_.push_back(consumer);
   }
 
@@ -123,7 +123,7 @@ class MultiConsumer : public IConsumer<T>
    */
   void removeConsumer(std::shared_ptr<IConsumer<T>> consumer)
   {
-    std::lock_guard<std::mutex> lk(consumer_list);
+    std::lock_guard<std::mutex> lk(consumer_list_);
     auto it = std::find(consumers_.begin(), consumers_.end(), consumer);
     if (it == consumers_.end())
     {
@@ -183,7 +183,7 @@ class MultiConsumer : public IConsumer<T>
    */
   bool consume(std::shared_ptr<T> product)
   {
-    std::lock_guard<std::mutex> lk(consumer_list);
+    std::lock_guard<std::mutex> lk(consumer_list_);
     bool res = true;
     for (auto& con : consumers_)
     {
@@ -194,7 +194,7 @@ class MultiConsumer : public IConsumer<T>
   }
 
 private:
-  std::mutex consumer_list;
+  std::mutex consumer_list_;
 };
 
 /*!

include/ur_client_library/primary/primary_consumer.h

@@ -119,7 +119,7 @@ class PrimaryConsumer : public AbstractPrimaryConsumer
     code.timestamp = pkg.timestamp_;
     code.to_string = pkg.toString();
 
-    const auto logContents = "Logging an ErrorCodeMessage from the UR Controller Box: " + pkg.toString();
+    const auto log_contents = "Logging an ErrorCodeMessage from the UR Controller Box: " + pkg.toString();
 
     switch (code.report_level)
     {
@@ -129,18 +129,18 @@ class PrimaryConsumer : public AbstractPrimaryConsumer
       case urcl::primary_interface::ReportLevel::DEVL_WARNING:
       case urcl::primary_interface::ReportLevel::DEVL_VIOLATION:
       case urcl::primary_interface::ReportLevel::DEVL_FAULT:
-        URCL_LOG_DEBUG(logContents.c_str());
+        URCL_LOG_DEBUG(log_contents.c_str());
         break;
       case urcl::primary_interface::ReportLevel::INFO:
-        URCL_LOG_INFO(logContents.c_str());
+        URCL_LOG_INFO(log_contents.c_str());
         break;
       case urcl::primary_interface::ReportLevel::WARNING:
-        URCL_LOG_WARN(logContents.c_str());
+        URCL_LOG_WARN(log_contents.c_str());
         break;
       default:
         // urcl::primary_interface::ReportLevel::VIOLATION:
         // urcl::primary_interface::ReportLevel::FAULT:
-        URCL_LOG_ERROR(logContents.c_str());
+        URCL_LOG_ERROR(log_contents.c_str());
         break;
     }