Skip to content

Commit d0cc3a8

Browse files
authored
WiFiServer - don't inherit from Server class (#8418)
* WiFiServer - don't inherit from Server class
1 parent dde2c76 commit d0cc3a8

File tree

4 files changed

+2
-29
lines changed

4 files changed

+2
-29
lines changed

libraries/ESP8266WiFi/src/ArduinoWiFiServer.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
#endif
2929

3030
template <class TServer, class TClient>
31-
class ArduinoCompatibleWiFiServerTemplate : public TServer {
31+
class ArduinoCompatibleWiFiServerTemplate : public TServer, public Print {
3232
public:
3333

3434
ArduinoCompatibleWiFiServerTemplate(const IPAddress& addr, uint16_t port) : TServer(addr, port) {}

libraries/ESP8266WiFi/src/WiFiServer.cpp

-12
Original file line numberDiff line numberDiff line change
@@ -171,18 +171,6 @@ void WiFiServer::stop() {
171171
close();
172172
}
173173

174-
size_t WiFiServer::write(uint8_t b) {
175-
return write(&b, 1);
176-
}
177-
178-
size_t WiFiServer::write(const uint8_t *buffer, size_t size) {
179-
// write to all clients
180-
// not implemented
181-
(void) buffer;
182-
(void) size;
183-
return 0;
184-
}
185-
186174
template<typename T>
187175
T* slist_append_tail(T* head, T* item) {
188176
if (!head)

libraries/ESP8266WiFi/src/WiFiServer.h

+1-4
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ extern "C" {
6565
class ClientContext;
6666
class WiFiClient;
6767

68-
class WiFiServer : public Server {
68+
class WiFiServer {
6969
// Secure server needs access to all the private entries here
7070
protected:
7171
uint16_t _port;
@@ -94,14 +94,11 @@ class WiFiServer : public Server {
9494
void begin(uint16_t port, uint8_t backlog);
9595
void setNoDelay(bool nodelay);
9696
bool getNoDelay();
97-
virtual size_t write(uint8_t);
98-
virtual size_t write(const uint8_t *buf, size_t size);
9997
uint8_t status();
10098
uint16_t port() const;
10199
void close();
102100
void stop();
103101

104-
using Print::write;
105102
using ClientType = WiFiClient;
106103

107104
protected:

tests/host/common/MockWiFiServerSocket.cpp

-12
Original file line numberDiff line numberDiff line change
@@ -127,18 +127,6 @@ bool WiFiServer::hasClient ()
127127
return poll(&p, 1, 0) && p.revents == POLLIN;
128128
}
129129

130-
size_t WiFiServer::write (uint8_t c)
131-
{
132-
return write(&c, 1);
133-
}
134-
135-
size_t WiFiServer::write (const uint8_t *buf, size_t size)
136-
{
137-
fprintf(stderr, MOCK "todo: WiFiServer::write(%p, %zd)\n", buf, size);
138-
exit(EXIT_FAILURE);
139-
return 0;
140-
}
141-
142130
void WiFiServer::close ()
143131
{
144132
if (pcb2int(_listen_pcb) >= 0)

0 commit comments

Comments
 (0)