-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Bring PR #31776(Adjust storage for large payloads) from tcp branch to master #33001
Conversation
PR #33001: Size comparison from 4aadee7 to d862fa3 Increases (49 builds for bl602, bl702, bl702l, cc13x4_26x4, linux, mbed, nrfconnect, qpg, stm32, telink)
Decreases (44 builds for bl702, bl702l, linux, nrfconnect, nxp, qpg, stm32, telink)
Full report (67 builds for bl602, bl702, bl702l, cc13x4_26x4, linux, mbed, nrfconnect, nxp, qpg, stm32, telink)
|
PR #33001: Size comparison from bfdb5da to fb8863b Increases (59 builds for bl602, bl702, cc13x4_26x4, cc32xx, efr32, esp32, linux, mbed, nrfconnect, psoc6, qpg, stm32, telink)
Decreases (48 builds for bl702, bl702l, cyw30739, linux, nrfconnect, nxp, qpg, stm32, telink)
Full report (82 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
ba8bf3c
to
bcf5e08
Compare
PR #33001: Size comparison from 0a87dca to 4800bc6 Increases (60 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, efr32, esp32, linux, mbed, nrfconnect, psoc6, qpg, stm32, telink)
Decreases (27 builds for bl702, bl702l, cyw30739, linux, nxp, telink)
Full report (82 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
PR #33001: Size comparison from 35ced57 to d6d0b8d Increases (60 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, efr32, esp32, linux, mbed, nrfconnect, psoc6, qpg, stm32, telink)
Decreases (27 builds for bl702, bl702l, cyw30739, linux, nxp, telink)
Full report (82 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
PR #33001: Size comparison from 01facfd to b436e06 Increases (6 builds for cc32xx, mbed, qpg, stm32)
Full report (6 builds for cc32xx, mbed, qpg, stm32)
|
PR #33001: Size comparison from c6fbf7c to 11c9d5c Increases (59 builds for bl602, bl702, cc13x4_26x4, cc32xx, efr32, esp32, linux, mbed, nrfconnect, psoc6, qpg, stm32, telink)
Decreases (28 builds for bl702, bl702l, cyw30739, linux, nxp, telink)
Full report (82 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
PR #33001: Size comparison from 511c974 to fd749b2 Increases (56 builds for bl602, bl702, cc13x4_26x4, efr32, esp32, linux, mbed, nrfconnect, psoc6, qpg, telink)
Decreases (29 builds for bl702, bl702l, cyw30739, efr32, linux, nxp, telink)
Full report (79 builds for bl602, bl702, bl702l, cc13x4_26x4, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, telink)
|
PR #33001: Size comparison from 8a4dffc to 2a6ea2f Increases (59 builds for bl602, bl702, cc13x4_26x4, cc32xx, efr32, esp32, linux, mbed, nrfconnect, psoc6, qpg, stm32, telink)
Decreases (30 builds for bl702, bl702l, cyw30739, efr32, linux, nxp, telink)
Full report (82 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
a19af1a
to
beeaa21
Compare
and associated code for large payloads. Extend uint16_t type variables to size_t for the APIs and all applicable places.
PR #33001: Size comparison from a28965b to aca8ea7 Increases (59 builds for bl602, bl702, cc13x4_26x4, cc32xx, efr32, esp32, linux, mbed, nrfconnect, psoc6, qpg, stm32, telink)
Decreases (29 builds for bl702, bl702l, cyw30739, linux, nxp, telink)
Full report (82 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR is to bring PR#31776(in tcp branch) to master with the later review comments addressed.