WiFi: partially fix flush() behaviour

cmaglie · cmaglie · commit 4878d5c2ecf9 · 2015-11-11T16:31:46.000+01:00
We still need to check if trasmission is completed, BTW this commit should alleviate bugs involving data loss. See #4029
diff --git a/libraries/WiFi/src/WiFiClient.cpp b/libraries/WiFi/src/WiFiClient.cpp
@@ -140,8 +140,7 @@ int WiFiClient::peek() {
 }
 
 void WiFiClient::flush() {
-  while (available())
-    read();
+  // TODO: a real check to ensure transmission has been completed
 }
 
 void WiFiClient::stop() {
diff --git a/libraries/WiFi/src/WiFiUdp.cpp b/libraries/WiFi/src/WiFiUdp.cpp
@@ -155,8 +155,7 @@ int WiFiUDP::peek()
 
 void WiFiUDP::flush()
 {
-  while (available())
-    read();
+  // TODO: a real check to ensure transmission has been completed
 }
 
 IPAddress  WiFiUDP::remoteIP()

Original file line number	Diff line number	Diff line change
`@@ -140,8 +140,7 @@ int WiFiClient::peek() {`
`140`	`140`	`}`
`141`	`141`
`142`	`142`	`void WiFiClient::flush() {`
`143`		`- while (available())`
`144`		`- read();`
	`143`	`+ // TODO: a real check to ensure transmission has been completed`
`145`	`144`	`}`
`146`	`145`
`147`	`146`	`void WiFiClient::stop() {`
Original file line number	Diff line number	Diff line change
`@@ -155,8 +155,7 @@ int WiFiUDP::peek()`
`155`	`155`
`156`	`156`	`void WiFiUDP::flush()`
`157`	`157`	`{`
`158`		`- while (available())`
`159`		`- read();`
	`158`	`+ // TODO: a real check to ensure transmission has been completed`
`160`	`159`	`}`
`161`	`160`
`162`	`161`	`IPAddress WiFiUDP::remoteIP()`