Eliminate dedicated status variable is_reader because checking whether or not the receive ringbuffer has been allocated contains the same status information.

Author: aentinger

src/serial/SerialDispatcher.cpp

@@ -225,7 +225,6 @@ std::list<SerialDispatcher::ThreadCustomerData>::iterator SerialDispatcher::find
 
 void SerialDispatcher::prepareSerialReader(std::list<ThreadCustomerData>::iterator & iter)
 {
-  iter->is_reader = true;
   if (!iter->rx_buffer)
     iter->rx_buffer.reset(new arduino::RingBuffer());
 }
@@ -240,7 +239,7 @@ void SerialDispatcher::handleSerialReader()
                   std::end  (_thread_customer_list),
                   [c](ThreadCustomerData & d)
                   {
-                    if (!d.is_reader)
+                    if (!d.rx_buffer)
                       return;
 
                     if (!d.rx_buffer->availableForStore())

src/serial/SerialDispatcher.h

@@ -74,7 +74,6 @@ class SerialDispatcher : public arduino::HardwareSerial
     osThreadId_t thread_id;
     arduino::RingBuffer tx_buffer;
     bool block_tx_buffer;
-    bool is_reader;                                 /* This thread has expressed interest to read from Serial via a call to either read() or available(). */
     mbed::SharedPtr<arduino::RingBuffer> rx_buffer; /* Only when a thread has expressed interested to read from serial a receive ringbuffer is allocated. */
   } ThreadCustomerData;