Merge pull request #136 from sparkfun/release_candidate

v2.2.9 - add a destructor to resolve issue #135

Author: PaulZC

library.properties

@@ -1,5 +1,5 @@
 name=SparkFun u-blox GNSS Arduino Library
-version=2.2.8
+version=2.2.9
 author=SparkFun Electronics <[email protected]>
 maintainer=SparkFun Electronics <sparkfun.com>
 sentence=Library for I2C, Serial and SPI Communication with u-blox GNSS modules<br/><br/>

src/SparkFun_u-blox_GNSS_Arduino_Library.cpp

@@ -63,13 +63,40 @@ SFE_UBLOX_GNSS::SFE_UBLOX_GNSS(void)
 #endif
 }
 
+SFE_UBLOX_GNSS::~SFE_UBLOX_GNSS(void)
+{
+  // Destructor
+
+  end(); // Delete all allocated memory - excluding payloadCfg, payloadAuto and spiBuffer
+
+  if (payloadCfg != NULL)
+  {
+    delete[] payloadCfg; // Created with new[]
+    payloadCfg = NULL;   // Redundant?
+  }
+
+  if (payloadAuto != NULL)
+  {
+    delete[] payloadAuto; // Created with new[]
+    payloadAuto = NULL;   // Redundant?
+  }
+
+  if (spiBuffer != NULL)
+  {
+    delete[] spiBuffer; // Created with new[]
+    spiBuffer = NULL;   // Redundant?
+  }
+}
+
 // Stop all automatic message processing. Free all used RAM
 void SFE_UBLOX_GNSS::end(void)
 {
   // Note: payloadCfg is not deleted
 
   // Note: payloadAuto is not deleted
 
+  // Note: spiBuffer is not deleted
+
   if (ubxFileBuffer != NULL) // Check if RAM has been allocated for the file buffer
   {
 #ifndef SFE_UBLOX_REDUCED_PROG_MEM

src/SparkFun_u-blox_GNSS_Arduino_Library.h

@@ -643,6 +643,7 @@ class SFE_UBLOX_GNSS
 {
 public:
   SFE_UBLOX_GNSS(void);
+  ~SFE_UBLOX_GNSS(void);
 
 // A default of 250ms for maxWait seems fine for I2C but is not enough for SerialUSB.
 // If you know you are only going to be using I2C / Qwiic communication, you can