Skip to content

gnrc_tcp: Add GNRC_TCP_NO_TIMEOUT#16735

Merged
benpicco merged 1 commit intoRIOT-OS:masterfrom
brummer-simon:gnrc_tcp-add_GNRC_TCP_NO_TIMEOUT
Aug 14, 2021
Merged

gnrc_tcp: Add GNRC_TCP_NO_TIMEOUT#16735
benpicco merged 1 commit intoRIOT-OS:masterfrom
brummer-simon:gnrc_tcp-add_GNRC_TCP_NO_TIMEOUT

Conversation

@brummer-simon
Copy link
Member

Contribution description

This PR adds the constant GNRC_TCP_NO_TIMEOUT to disable user specified timeouts on the functions gnrc_tcp_accept, gnrc_tcp_send, gnrc_tcp_recv.

Testing procedure

This PR can be partially tested by executing the tests under tests/gnrc_tcp. Tests exist for gnrc_tcp_accept and gnrc_tcp_recv.
The function gnrc_tcp_send can't be tested because as soon as a connection was established the host system stores received data, therefore gnrc_tcp_send would be unblocked.

Issues/PRs references

Adds missing functionality required by #16494

@github-actions github-actions bot added Area: network Area: Networking Area: sys Area: System Area: tests Area: tests and testing framework labels Aug 14, 2021
Copy link
Contributor

@benpicco benpicco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI has some style nits

@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Aug 14, 2021
@benpicco benpicco merged commit 84413a3 into RIOT-OS:master Aug 14, 2021
@benpicco benpicco added this to the Release 2021.10 milestone Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: network Area: Networking Area: sys Area: System Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants