WiFiTelnetToSerial Example - Minor Issues

[nameoftherose]

Reference is made to WiFiTelnetToSerial.ino
The loop in lines 55-63 continues even if an available client slot has been found. As a result all empty spots are allocated and spurious New Client messages are produced. I propose that the continue statement in line 61 is replaced by a break.
The statements in lines 65-66 are executed always even if a client spot was previously allocated. As a result, a race condition may arise and an incoming connection rejected. I propose that these statements are executed only when i==MAX_SRV_CLIENTS ie said lines 65-66 are replaced with:

if (i==MAX_SRV_CLIENTS) {
      WiFiClient serverClient = server.available();
      serverClient.stop();
}

[devyte]

@nameoftherose The code you reference is still there in latest git.
Your changes seem sane to me. Did you test these modifications? If so, could you please make a PR?

[nameoftherose]

@devyte I am sorry for the delayed response. Indeed I have tested these modifications. I did Pull Request #4256 as requested. Thank You.