Merge 7f430ce6bd into d46a6f092a

Wifi_NINA.cpp

@@ -122,8 +122,6 @@ bool WifiNINA::setup(const char *SSid,
       DIAG(F("Forcing one more Wifi restart"));
       // esp_wifi_start();
       // esp_wifi_connect();
-      WiFi.end();
-      WiFi.begin(SSid, password);
       tries=40;
       while (WiFi.status() != WL_CONNECTED && tries) {
         Serial.print('.');
@@ -221,7 +219,8 @@ void WifiNINA::checkForNewClient() {
   for (byte clientId=0; clientId<MAX_CLIENTS; clientId++){
     if (!clients[clientId]) {
       clients[clientId]= new WiFiClient(newClient); // use this slot
-      //DIAG(F("New client connected to slot %d"),clientId); //TJF: brought in for debugging.
+      clients[clientId]->flush(); // clear out the input buffer
+      DIAG(F("New client connected to slot %d"),clientId); //TJF: brought in for debugging.
       return;
     }
   }
@@ -232,8 +231,9 @@ void WifiNINA::checkForLostClients() {
     auto c=clients[clientId];
     if(c && !c->connected()) {
       clients[clientId]->stop();
-      //DIAG(F("Remove client %d"), clientId);
+      DIAG(F("Remove client %d"), clientId);
       CommandDistributor::forget(clientId);
+      //delete c; //TJF: this causes a crash when client drops.. commenting out for now.
       clients[clientId]=nullptr;
     }
   }
@@ -266,11 +266,16 @@ void WifiNINA::checkForClientOutput() {
   if (!c) {
     // client is gone, throw away msg
     for (int i=0;i<replySize;i++) outboundRing->read();
-    //DIAG(F("gone, drop message.")); //TJF: only for diag
+    DIAG(F("gone, drop message.")); //TJF: only for diag
     return;
   }
   // emit data to the client object
+  // This should work in theory, the
+  //DIAG(F("send message")); //TJF: only for diag
+  //TJF: the old code had to add a 0x00 byte to the end to terminate the
+  //TJF: c string, before sending it. i take it this is not needed?
   for (int i=0;i<replySize;i++) c->write(outboundRing->read());
+  //c->write((byte)0x00);
 }
 
 void WifiNINA::loop() {