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.

Sign up for GitHub

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

Jump to bottom

C1: Fix-up CHIPClusters.cpp to internally use SingleInvokeInitiator #8958

Closed
mrjerryjohns opened this issue Aug 12, 2021 · 0 comments · Fixed by #9877
Closed

C1: Fix-up CHIPClusters.cpp to internally use SingleInvokeInitiator #8958

mrjerryjohns opened this issue Aug 12, 2021 · 0 comments · Fixed by #9877
Assignees
@erjiaqing
Labels
Interaction Model Work

Comments

@mrjerryjohns
Copy link
Contributor

Problem

Fix the generated logic in CHIPClusters.cpp to internally use SingleInvokeInitiator instead of CommandSender without changing its public APIs.

This permits validation of the new SingleInvokeInitiator class without actually changing the various app call-sites.

As part of this fix, the internal guts of that logic should now not have to use any of the gCallbackMgr logic.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@erjiaqing erjiaqing

Labels
Interaction Model Work
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[IM] Remove CommandSender object pool in interaction model engine erjiaqing/connectedhomeip
2 participants
@erjiaqing @mrjerryjohns