Update comments

Author: PaulZC

src/u-blox_GNSS.cpp

@@ -5825,6 +5825,7 @@ bool DevUBLOXGNSS::pushRawData(uint8_t *dataBytes, size_t numDataBytes, bool cal
     return false; // Indicate to the user that there was no data to push
 
   if (!lock()) return false;
+
   bool ok = false;
   if (_commType == COMM_TYPE_SERIAL)
   {
@@ -5932,7 +5933,9 @@ bool DevUBLOXGNSS::pushRawData(uint8_t *dataBytes, size_t numDataBytes, bool cal
 
     ok = true;
   }
+
   unlock();
+  
   return ok;
 }
 

src/u-blox_GNSS.h

@@ -136,13 +136,14 @@ class DevUBLOXGNSS
   // Flag to indicate if we are connected to UART1 or UART2
   // Needed to select the correct config items when enabling a periodic message
   bool _UART2 = false; // Default to UART1
-  
-  // the lock / unlock functions can be used if you have multiple tasks writing to the bus. 
-  // the idea is that in a RTOS you override this class and the two functions in which you take and give a mutex.
+
+  // These lock / unlock functions can be used if you have multiple tasks writing to the bus. 
+  // The idea is that in a RTOS you override this class and the two functions in which you take and give a mutex.
   virtual bool lock(void) { return true; }
   virtual void unlock(void) { }
 
-  // A simpler lock to prevent checkUblox from being called while a sendCommand and waitForResponse is in progress
+  // A simpler lock to prevent checkUblox from being called by another thread while a full sendCommand and waitForResponse is in progress.
+  // Note: this won't prevent pushRawData from gatecrashing the party...
   void lockCheckUblox(void)  __attribute__((weak));
   void unlockCheckUblox(void)  __attribute__((weak));
   volatile bool _checkUbloxLock = false;