Improve mutex lock logic.

[apps/low-level-can-service.git] / src / can_reader.cpp

diff --git a/src/can_reader.cpp b/src/can_reader.cpp
index e2f6277..7ae981b 100644 (file)
--- a/src/can_reader.cpp
+++ b/src/can_reader.cpp
@@ -27,17 +27,12 @@ void can_reader(can_bus_dev_t &can_bus_dev, can_bus_t& can_bus)
 
        while(can_bus_dev.is_running())
        {
-               /* Declare and take lock ownership of can_frame_mutex.
-                * then waiting notification for a new can frame arrival
-                */
-               std::unique_lock<std::mutex> can_frame_lock(can_frame_mutex);
-               new_can_frame.wait(can_frame_lock);
-                       can_message.convert_from_canfd_frame(can_bus_dev.read());
-               can_frame_mutex.unlock();
+               can_message.convert_from_canfd_frame(can_bus_dev.read());
 
-               std::lock_guard<std::mutex> can_message_lock(can_message_mutex);
+               {
+                       std::lock_guard<std::mutex> can_message_lock(can_message_mutex);
                        can_bus.push_new_can_message(can_message);
-               can_message_mutex.unlock();
+               }
                new_can_message.notify_one();
        }
 }
\ No newline at end of file