-
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
Group MRP parameters #12069
Group MRP parameters #12069
Conversation
(#6652): This is a stub implementation, should be replaced by the real one when CASE and PASE is completedconnectedhomeip/src/transport/PairingSession.h Lines 92 to 98 in 105366d
This comment was generated by todo based on a
|
PR #12069: Size comparison from 55349a5 to 105366d Increases above 0.2%:
Increases (17 builds for efr32, linux, p6, qpg, telink)
Decreases (11 builds for linux, p6, qpg)
Full report (18 builds for efr32, linux, p6, qpg, telink)
|
PR #12069: Size comparison from 55349a5 to 8f22514 Increases above 0.2%:
Increases (35 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (21 builds for esp32, k32w, linux, mbed, nrfconnect, p6, qpg)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
@kghost - one comment left (minor) and merge conflicts |
move following 5 functions into SessionHandle once we can access session vars w/o using a SessionManagerconnectedhomeip/src/messaging/ExchangeContext.h Lines 169 to 172 in ae9a662
This comment was generated by todo based on a
|
PR #12069: Size comparison from 71a3fb2 to 92435e1 Increases above 0.2%:
Increases (35 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (20 builds for esp32, k32w, linux, mbed, nrfconnect, p6, qpg)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Problem
Passing MRP parameters as 2 int is hard to maintain.
Change overview
Group MRP parameters into
ReliableMessageProtocolConfig
classTesting
Verified by unit-tests