-
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
Simplfy the implementation of Mode Select Cluster #11359
Conversation
Configure your options for each endpointconnectedhomeip/src/app/clusters/mode-select-server/static-supported-modes-manager.cpp Lines 26 to 36 in fda93f9
This comment was generated by todo based on a
|
examples/all-clusters-app/ameba/main/include/static-supported-modes-manager.h
Outdated
Show resolved
Hide resolved
examples/all-clusters-app/ameba/main/static-supported-modes-manager.cpp
Outdated
Show resolved
Hide resolved
51a9328
to
c6c2d4b
Compare
PR #11359: Size comparison from 86511a1 to c6c2d4b Increases (2 builds for mbed, p6)
Decreases (5 builds for esp32, linux, mbed, p6)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
c6c2d4b
to
823abda
Compare
PR #11359: Size comparison from a04576d to 823abda Increases (1 build for p6)
Decreases (1 build for p6)
Full report (9 builds for k32w, p6, qpg, telink)
|
PR #11359: Size comparison from a04576d to 2aeede1 Increases (2 builds for mbed, p6)
Decreases (5 builds for esp32, linux, mbed, p6)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
2aeede1
to
499bbaf
Compare
499bbaf
to
4b75caa
Compare
PR #11359: Size comparison from 94c5144 to 4b75caa Increases (2 builds for mbed, p6)
Decreases (5 builds for esp32, linux, mbed, p6)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
4b75caa
to
dc21b2e
Compare
PR #11359: Size comparison from 372dc6a to dc21b2e Increases (1 build for p6)
Decreases (2 builds for linux, p6)
Full report (21 builds for efr32, k32w, linux, p6, qpg, telink)
|
abe4379
to
f3a42e9
Compare
PR #11359: Size comparison from 372dc6a to f3a42e9 Increases (2 builds for mbed, p6)
Decreases (5 builds for esp32, linux, mbed, p6)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Restyled by whitespace Restyled by clang-format Update submodules
f3a42e9
to
327b91b
Compare
PR #11359: Size comparison from ff5f946 to 327b91b Increases (2 builds for mbed, p6)
Decreases (5 builds for esp32, linux, mbed, p6)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Fast tracking as this has had > 3 days to review, and comments are addressed |
Restyled by whitespace Restyled by clang-format Update submodules
Problem
What is being fixed? Examples:
Change overview
Removed the usage of std::map
Removed the abstract ModeOptionIterator and its implementation
Removed the abstract ModeOptionIteratorFactory and its implementation
Move the implementation of StaticSupportedModeOptionsManager to /example folders, as it contains app-specific logic.
Added the concrete class ModeOptionsProvider
Testing
How was this tested? (at least one bullet point required)
Manually tested on MacOs with chip-tool