Move mktime call to WiFiClass::getTime, to prevent linking all the time

Author: sandeepmistry

src/WiFi.cpp

@@ -164,18 +164,11 @@ static void wifi_cb(uint8_t u8MsgType, void *pvMsg)
 
 		case M2M_WIFI_RESP_GET_SYS_TIME:
 		{
-			struct tm tm;
-			tstrSystemTime *pstrSystemTime = (tstrSystemTime *)pvMsg;
-
-			tm.tm_year = pstrSystemTime->u16Year - 1900;
-			tm.tm_mon = pstrSystemTime->u8Month - 1;
-			tm.tm_mday = pstrSystemTime->u8Day;
-			tm.tm_hour = pstrSystemTime->u8Hour;
-			tm.tm_min = pstrSystemTime->u8Minute;
-			tm.tm_sec = pstrSystemTime->u8Second;
-			tm.tm_isdst = -1;
-
-			WiFi._resolve = mktime(&tm);
+			if (WiFi._resolve != 0) {
+				memcpy((tstrSystemTime *)WiFi._resolve, pvMsg, sizeof(tstrSystemTime));
+			}
+
+			WiFi._resolve = 0;
 		}
 		break;
 
@@ -912,15 +905,33 @@ int WiFiClass::ping(IPAddress host, uint8_t ttl)
 
 uint32_t WiFiClass::getTime()
 {
-	_resolve = (uint32_t)-1;
+	tstrSystemTime systemTime;
+
+	_resolve = (uint32_t)&systemTime;
 
 	m2m_wifi_get_sytem_time();
 
 	unsigned long start = millis();
-	while (_resolve == ((uint32_t)-1) && millis() - start < 5000) {
+	while (_resolve != 0 && millis() - start < 5000) {
 		m2m_wifi_handle_events(NULL);
 	}
 
+	if (_resolve == 0) {
+		struct tm tm;
+
+		tm.tm_year = systemTime.u16Year - 1900;
+		tm.tm_mon = systemTime.u8Month - 1;
+		tm.tm_mday = systemTime.u8Day;
+		tm.tm_hour = systemTime.u8Hour;
+		tm.tm_min = systemTime.u8Minute;
+		tm.tm_sec = systemTime.u8Second;
+		tm.tm_isdst = -1;
+
+		_resolve = mktime(&tm);
+	} else {
+		_resolve = (uint32_t)-1;
+	}
+
 	return _resolve;
 }