Skip to content

Commit 91e2ab1

Browse files
committed
WiFiServer - 'rename' available() to accept()
1 parent 42aa0e6 commit 91e2ab1

File tree

6 files changed

+18
-8
lines changed

6 files changed

+18
-8
lines changed

libraries/ESP8266WiFi/src/ArduinoWiFiServer.h

+1-6
Original file line numberDiff line numberDiff line change
@@ -35,11 +35,6 @@ class ArduinoCompatibleWiFiServerTemplate : public TServer {
3535
ArduinoCompatibleWiFiServerTemplate(uint16_t port) : TServer(port) {}
3636
virtual ~ArduinoCompatibleWiFiServerTemplate() {}
3737

38-
// https://www.arduino.cc/en/Reference/EthernetServerAccept
39-
TClient accept() {
40-
return TServer::available();
41-
}
42-
4338
// https://www.arduino.cc/en/Reference/WiFiServerAvailable
4439
TClient available() {
4540

@@ -132,7 +127,7 @@ class ArduinoCompatibleWiFiServerTemplate : public TServer {
132127
void acceptClients() {
133128
for (uint8_t i = 0; i < MAX_MONITORED_CLIENTS; i++) {
134129
if (!connectedClients[i]) {
135-
connectedClients[i] = accept();
130+
connectedClients[i] = TServer::accept();
136131
}
137132
}
138133
}

libraries/ESP8266WiFi/src/WiFiServer.cpp

+4
Original file line numberDiff line numberDiff line change
@@ -130,6 +130,10 @@ bool WiFiServer::hasMaxPendingClients() {
130130

131131
WiFiClient WiFiServer::available(byte* status) {
132132
(void) status;
133+
return accept();
134+
}
135+
136+
WiFiClient WiFiServer::accept() {
133137
if (_unclaimed) {
134138
WiFiClient result(_unclaimed);
135139

libraries/ESP8266WiFi/src/WiFiServer.h

+2-1
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,8 @@ class WiFiServer : public Server {
8080
WiFiServer(const IPAddress& addr, uint16_t port);
8181
WiFiServer(uint16_t port);
8282
virtual ~WiFiServer() {}
83-
WiFiClient available(uint8_t* status = NULL);
83+
WiFiClient accept(); // https://www.arduino.cc/en/Reference/EthernetServerAccept
84+
WiFiClient available(uint8_t* status = NULL) __attribute__((deprecated("Renamed to accept().")));
8485
bool hasClient();
8586
// hasClientData():
8687
// returns the amount of data available from the first client

libraries/ESP8266WiFi/src/WiFiServerSecureBearSSL.cpp

+4
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,10 @@ void WiFiServerSecure::setECCert(const X509List *chain, unsigned cert_issuer_key
7777
// then any validation (i.e. client cert checking) will have succeeded.
7878
WiFiClientSecure WiFiServerSecure::available(uint8_t* status) {
7979
(void) status; // Unused
80+
return accept();
81+
}
82+
83+
WiFiClientSecure WiFiServerSecure::accept() {
8084
if (_unclaimed) {
8185
if (_sk && _sk->isRSA()) {
8286
WiFiClientSecure result(_unclaimed, _chain, _sk, _iobuf_in_size, _iobuf_out_size, _cache, _client_CA_ta, _tls_min, _tls_max);

libraries/ESP8266WiFi/src/WiFiServerSecureBearSSL.h

+2-1
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,8 @@ class WiFiServerSecure : public WiFiServer {
6565
bool setSSLVersion(uint32_t min = BR_TLS10, uint32_t max = BR_TLS12);
6666

6767
// If awaiting connection available and authenticated (i.e. client cert), return it.
68-
WiFiClientSecure available(uint8_t* status = NULL);
68+
WiFiClientSecure accept(); // https://www.arduino.cc/en/Reference/EthernetServerAccept
69+
WiFiClientSecure available(uint8_t* status = NULL) __attribute__((deprecated("Renamed to accept().")));
6970

7071
WiFiServerSecure& operator=(const WiFiServerSecure&) = default;
7172

tests/host/common/MockWiFiServer.cpp

+5
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,11 @@ WiFiServer::WiFiServer (uint16_t port)
5858
WiFiClient WiFiServer::available (uint8_t* status)
5959
{
6060
(void)status;
61+
return accept();
62+
}
63+
64+
WiFiClient WiFiServer::accept ()
65+
{
6166
if (hasClient())
6267
return WiFiClient(new ClientContext(serverAccept(pcb2int(_listen_pcb))));
6368
return WiFiClient();

0 commit comments

Comments
 (0)