Merge pull request #179 from cmumford/fix-protected

Corrected client public/private methods.

Author: brentru

Adafruit_MQTT_Client.h

@@ -41,11 +41,14 @@ class Adafruit_MQTT_Client : public Adafruit_MQTT {
                        const char *user = "", const char *pass = "")
       : Adafruit_MQTT(server, port, user, pass), client(client) {}
 
-  bool connectServer();
-  bool disconnectServer();
-  bool connected();
-  uint16_t readPacket(uint8_t *buffer, uint16_t maxlen, int16_t timeout);
-  bool sendPacket(uint8_t *buffer, uint16_t len);
+  bool connected() override;
+
+protected:
+  bool connectServer() override;
+  bool disconnectServer() override;
+  uint16_t readPacket(uint8_t *buffer, uint16_t maxlen,
+                      int16_t timeout) override;
+  bool sendPacket(uint8_t *buffer, uint16_t len) override;
 
 private:
   Client *client;

Adafruit_MQTT_FONA.h

@@ -42,7 +42,13 @@ class Adafruit_MQTT_FONA : public Adafruit_MQTT {
                      const char *user = "", const char *pass = "")
       : Adafruit_MQTT(server, port, user, pass), fona(f) {}
 
-  bool connectServer() {
+  bool connected() {
+    // Return true if connected, false if not connected.
+    return fona->TCPconnected();
+  }
+
+protected:
+  bool connectServer() override {
     char server[40];
     strncpy(server, servername, 40);
 #ifdef ADAFRUIT_SLEEPYDOG_H
@@ -54,14 +60,10 @@ class Adafruit_MQTT_FONA : public Adafruit_MQTT {
     return fona->TCPconnect(server, portnum);
   }
 
-  bool disconnectServer() { return fona->TCPclose(); }
-
-  bool connected() {
-    // Return true if connected, false if not connected.
-    return fona->TCPconnected();
-  }
+  bool disconnectServer() override { return fona->TCPclose(); }
 
-  uint16_t readPacket(uint8_t *buffer, uint16_t maxlen, int16_t timeout) {
+  uint16_t readPacket(uint8_t *buffer, uint16_t maxlen,
+                      int16_t timeout) override {
     uint8_t *buffp = buffer;
     DEBUG_PRINTLN(F("Reading data.."));
 
@@ -114,7 +116,7 @@ class Adafruit_MQTT_FONA : public Adafruit_MQTT {
     return len;
   }
 
-  bool sendPacket(uint8_t *buffer, uint16_t len) {
+  bool sendPacket(uint8_t *buffer, uint16_t len) override {
     DEBUG_PRINTLN(F("Writing packet"));
     if (fona->TCPconnected()) {
       boolean ret = fona->TCPsend((char *)buffer, len);