-
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
Refactor/Split ExchangeMgrDelegate #11789
Conversation
this should be test only, after OnSessionReleased is move to SessionHandle within the exchange contextconnectedhomeip/src/messaging/ExchangeMgr.h Lines 251 to 256 in 7931e6d
This comment was generated by todo based on a
|
This is a temporary solution to release sessions, in the near future, SessionReleaseDelegate will beconnectedhomeip/src/transport/SessionManager.h Lines 272 to 279 in 7931e6d
This comment was generated by todo based on a
|
7931e6d
to
f929607
Compare
/rebase |
f929607
to
c404895
Compare
c404895
to
0e52d04
Compare
PR #11789: Size comparison from bbd19d9 to 0e52d04 Increases above 0.2%:
Increases (35 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (23 builds for esp32, linux, mbed, nrfconnect, p6)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
0e52d04
to
7acbf45
Compare
PR #11789: Size comparison from 4500190 to 7acbf45 Increases above 0.2%:
Increases (31 builds for efr32, esp32, k32w, linux, nrfconnect, p6, qpg, telink)
Decreases (19 builds for esp32, linux, nrfconnect, p6)
Full report (33 builds for efr32, esp32, k32w, linux, nrfconnect, p6, qpg, telink)
|
7acbf45
to
ffab23f
Compare
e1def42
to
a4b8d66
Compare
PR #11789: Size comparison from 0c88b57 to a4b8d66 Increases above 0.2%:
Increases (35 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (23 builds for esp32, linux, mbed, nrfconnect, p6)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
a4b8d66
to
9fd86ba
Compare
Update the MRP params based on the MRP params extracted from CASE, when this is available.connectedhomeip/src/app/OperationalDeviceProxy.cpp Lines 232 to 235 in 9fd86ba
This comment was generated by todo based on a
|
9fd86ba
to
10cd49d
Compare
PR #11789: Size comparison from cbd3e95 to 10cd49d Increases above 0.2%:
Increases (35 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (23 builds for esp32, linux, mbed, nrfconnect, p6)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Problem
SessionManageDelegate
andExchangeMgrDelegate
do not fit their role.Change overview
There are totally 4 callbacks in
SessionManageDelegate
:ExchangeMgrDelegate
duplicate 2 callbacks:This shouldn't be how these callback works.
In This PR, these callbacks will be split into 3 delegate classes:
SessionCreationDelegate::OnNewSession
: device controller is the only receiver now, exchange manager doesn't need this event.SessionReleaseDelegate::OnSessionReleased
: everything holding a session should be the receiver of this event, includingExchangeManager
,DeviceController
,SessionMessageDelegate::OnMessageReceived
:ExchangeManager
is the only receiver, all messages are handled by theExchangeManager
OnReceiveError
event is removed, we log all error path insideSessionManager
the event is not needed for applications.Testing
Verified using unit-tests.