[riot-notifications] [RIOT-OS/RIOT] lwip_sock: Make sock_tcp_read return more data if available and test it (#16302)

Martine Lenders notifications at github.com
Mon Apr 12 12:21:11 CEST 2021


@miri64 approved this pull request.

ACK, the change makes sense and the test still passes:

```console
$ RIOT_CI_BUILD=1 make -C tests/lwip -j flash test
make: Entering directory '/home/mlenders/Repositories/RIOT-OS/RIOT/tests/lwip'
Building application "tests_lwip" for "native" with MCU "native".

/usr/bin/ld: /home/mlenders/Repositories/RIOT-OS/RIOT/tests/lwip/bin/native/cpu/tramp.o: warning: relocation against `_native_saved_eip' in read-only section `.text'
/usr/bin/ld: warning: creating DT_TEXTREL in a PIE
   text	   data	    bss	    dec	    hex	filename
 234343	   1228	 101420	 336991	  5245f	/home/mlenders/Repositories/RIOT-OS/RIOT/tests/lwip/bin/native/tests_lwip.elf
Testing for (<Board 'native',port='tap0',serial=None>, <Board 'native',port='tap1',serial=None>): 
......
make: Leaving directory '/home/mlenders/Repositories/RIOT-OS/RIOT/tests/lwip'
$ git log -1 --oneline
9974803f43 (HEAD) tests/lwip: Add a test for partial TCP reads.
```



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/16302#pullrequestreview-633371606
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210412/52af05dc/attachment.htm>


More information about the notifications mailing list