Fix issue with creating temporary std::lock_gard which immediately re…

…leases lock
drnichols · Dec 3, 2023 · 5a0ea97 · 5a0ea97
1 parent 4808c0c
commit 5a0ea97
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/ecodan-ha-local/ehal_hp.cpp b/ecodan-ha-local/ehal_hp.cpp
@@ -278,7 +278,7 @@ namespace ehal::hp
  cmd.set_float16(newTemp, 10);
 
  {
- std::lock_guard<std::mutex>{cmdQueueMutex};
+ std::lock_guard<std::mutex> lock{cmdQueueMutex};
  cmdQueue.emplace(std::move(cmd));
  }
 
@@ -336,7 +336,7 @@ namespace ehal::hp
  cmd.set_float16(newTemp, 10);
 
  {
- std::lock_guard<std::mutex>{cmdQueueMutex};
+ std::lock_guard<std::mutex> lock{cmdQueueMutex};
  cmdQueue.emplace(std::move(cmd));
  }
 
@@ -368,7 +368,7 @@ namespace ehal::hp
  cmd.set_float16(newTemp, 8);
 
  {
- std::lock_guard<std::mutex>{cmdQueueMutex};
+ std::lock_guard<std::mutex> lock{cmdQueueMutex};
  cmdQueue.emplace(std::move(cmd));
  }
 
@@ -399,7 +399,7 @@ namespace ehal::hp
  cmd[5] = static_cast<uint8_t>(dhwMode);
 
  {
- std::lock_guard<std::mutex>{cmdQueueMutex};
+ std::lock_guard<std::mutex> lock{cmdQueueMutex};
  cmdQueue.emplace(std::move(cmd));
  }
 
@@ -419,7 +419,7 @@ namespace ehal::hp
  cmd[3] = on ? 1 : 0; // bit[3] of payload is DHW force, bit[2] is Holiday mode.
 
  {
- std::lock_guard<std::mutex>{cmdQueueMutex};
+ std::lock_guard<std::mutex> lock{cmdQueueMutex};
  cmdQueue.emplace(std::move(cmd));
  }
 
@@ -458,7 +458,7 @@ namespace ehal::hp
  cmd[6] = mode;
 
  {
- std::lock_guard<std::mutex>{cmdQueueMutex};
+ std::lock_guard<std::mutex> lock{cmdQueueMutex};
  cmdQueue.emplace(std::move(cmd));
  }