From da1cfc563e0ff9afda1953a61dff0800e61d8e90 Mon Sep 17 00:00:00 2001
From: PaulZC <pc235603@gmail.com>
Date: Mon, 28 Jun 2021 11:41:49 +0100
Subject: [PATCH] Add note to future self

---
 src/SparkFun_u-blox_GNSS_Arduino_Library.cpp | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/SparkFun_u-blox_GNSS_Arduino_Library.cpp b/src/SparkFun_u-blox_GNSS_Arduino_Library.cpp
index 6d2e434..55826c1 100644
--- a/src/SparkFun_u-blox_GNSS_Arduino_Library.cpp
+++ b/src/SparkFun_u-blox_GNSS_Arduino_Library.cpp
@@ -838,6 +838,8 @@ boolean SFE_UBLOX_GNSS::checkUbloxSpi(ubxPacket *incomingUBX, uint8_t requestedC
   _spiPort->beginTransaction(settingsA);
   digitalWrite(_csPin, LOW);
   uint8_t byteReturned = _spiPort->transfer(0x0A);
+  // Note to future self: I think the 0xFF check will cause problems when attempting to process (e.g.) RAWX data
+  // which could legitimately contain 0xFF within the data stream
   while (byteReturned != 0xFF || currentSentence != NONE)
   {       
     process(byteReturned, incomingUBX, requestedClass, requestedID);