Cleanup callback definition

Author: ilcato

src/ArduinoIoTCloud.cpp

@@ -206,10 +206,10 @@ void ArduinoIoTCloudClass::poll()
   update();
 }
 
-void ArduinoIoTCloudClass::update(void (*callback)(void))
+void ArduinoIoTCloudClass::update(CallbackFunc onSyncCompleteCallback)
 {
   // If user call update() without parameters use the default ones
-  update(MAX_RETRIES, RECONNECTION_TIMEOUT, callback);
+  update(MAX_RETRIES, RECONNECTION_TIMEOUT, onSyncCompleteCallback);
 }
 
 void ArduinoIoTCloudClass::update(int const reconnectionMaxRetries, int const reconnectionTimeoutMs)
@@ -237,8 +237,8 @@ void ArduinoIoTCloudClass::update(int const reconnectionMaxRetries, int const re
       }
       break;
     case SYNC_STATUS_VALUES_PROCESSED:
-      if(callback != NULL)
-        (*callback)();
+      if(onSyncCompleteCallback != NULL)
+        (*onSyncCompleteCallback)();
       syncStatus = SYNC_STATUS_SYNCHRONIZED;
       break;
   }

src/ArduinoIoTCloud.h

@@ -40,6 +40,8 @@ typedef struct {
   int timeout;
 } mqttConnectionOptions;
 
+typedef void (*CallbackFunc)(void);
+
 extern ConnectionManager *ArduinoIoTPreferredConnection;
 
 enum ArduinoIoTConnectionStatus {
@@ -77,10 +79,10 @@ class ArduinoIoTCloudClass {
   bool disconnect();
 
   void poll() __attribute__((deprecated)); /* Attention: Function is deprecated - use 'update' instead */
-  void update(void (*callback)(void) = NULL);
+  void update(CallbackFunc onSyncCompleteCallback = NULL);
 
   // defined for users who want to specify max reconnections reties and timeout between them
-  void update(int const reconnectionMaxRetries, int const reconnectionTimeoutMs, void (*callback)(void) = NULL);
+  void update(int const reconnectionMaxRetries, int const reconnectionTimeoutMs, CallbackFunc onSyncCompleteCallback = NULL);
 
   int connected();
   // Clean up existing Mqtt connection, create a new one and initialize it