esp8266 in _oob_* : recv() brought back where it was needed

Author: Dominika Maziec

components/wifi/esp8266-driver/ESP8266/ESP8266.cpp

@@ -686,7 +686,7 @@ void ESP8266::_oob_packet_hdlr()
     }
 
     if (_tcp_passive && _sock_i[id].open == true && _sock_i[id].proto == NSAPI_TCP) {
-        if (_parser.scanf("%d\n", &amount)) {
+        if (_parser.recv("%d\n", &amount)) {
             _sock_i[id].tcp_data_avbl = amount;
 
             // notify data is available

platform/ATCmdParser.h

@@ -271,6 +271,9 @@ class ATCmdParser : private NonCopyable<ATCmdParser> {
 
     /**
      * Direct scanf on underlying stream
+    * This function does not itself match whitespace in its format string, so \n is not significant to it.
+    * It should be used only when certain string is needed or format ends with certain character, otherwise
+    * it will fill the output with one character.
      * @see scanf
      *
      * @param format Format string to pass to scanf

platform/source/ATCmdParser.cpp

@@ -309,7 +309,6 @@ bool ATCmdParser::vrecv(const char *response, std::va_list args)
             if (whole_line_wanted && c != '\n') {
                 // Don't attempt scanning until we get delimiter if they included it in format
                 // This allows recv("Foo: %s\n") to work, and not match with just the first character of a string
-                // (scanf does not itself match whitespace in its format string, so \n is not significant to it)
             } else if (response) {
                 sscanf(_buffer + offset, _buffer, &count);
             }