Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove const qualifier from send/recv in transport interface #86

Merged
merged 2 commits into from
Oct 21, 2020

Conversation

yourslab
Copy link
Contributor

This follows changes from FreeRTOS/FreeRTOS@398abba. The const qualifier is removed from send/recv because there are transport implementations that require a member of the network context to be modified such as in the case of mbedtls.

muneebahmed10
muneebahmed10 previously approved these changes Oct 21, 2020
@yourslab yourslab merged commit 7f0478d into FreeRTOS:master Oct 21, 2020
yourslab added a commit to FreeRTOS/coreHTTP that referenced this pull request Oct 21, 2020
This follows changes from FreeRTOS/coreMQTT#86 & FreeRTOS/FreeRTOS@398abba. The const qualifier is removed from send/recv because there are transport implementations that require a member of the network context to be modified such as in the case of mbedtls.
yourslab added a commit to aws/aws-iot-device-sdk-embedded-C that referenced this pull request Oct 21, 2020
…d MQTT system test (#1326)

Submodule pointers are updated to follow changes from changes from FreeRTOS/coreMQTT#86, FreeRTOS/coreHTTP#45, & FreeRTOS/FreeRTOS@398abba. The const qualifier is removed from send/recv because there are transport implementations that require a member of the network context to be modified such as in the case of mbedtls.
yourslab added a commit to aws/amazon-freertos that referenced this pull request Oct 21, 2020
This follows changes from FreeRTOS/coreMQTT#86 & FreeRTOS/FreeRTOS@398abba. The const qualifier is removed from send/recv because there are transport implementations that require a member of the network context to be modified such as in the case of mbedtls.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants