-
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
Add initial codegen support for Cluster object #9971
Add initial codegen support for Cluster object #9971
Conversation
10cb962
to
b988b03
Compare
Need to add the BUILD.gn for the cluster-objects library as a dependency to all the existing sample apps. |
zzz_generated/lighting-app/zap-generated/IMClusterCommandHandler.cpp
Outdated
Show resolved
Hide resolved
b988b03
to
8a92045
Compare
8a92045
to
ddef21b
Compare
|
ddef21b
to
323ad8a
Compare
namespace clusters { | ||
|
||
namespace AccountLogin { | ||
constexpr ClusterId kClusterId = 0x50E; |
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.
This here is a duplication of information? zzz_generated/app-common/app-common/zap-generated/ids/Clusters.h
contains this information. Wouldn't it be better to reference this?
enum class ApplicationBasicStatus : uint8_t | ||
{ | ||
APPLICATION_BASIC_STATUS_STOPPED = 0x00, | ||
APPLICATION_BASIC_STATUS_ACTIVE_VISIBLE_FOCUS = 0x01, | ||
APPLICATION_BASIC_STATUS_ACTIVE_HIDDEN = 0x02, | ||
APPLICATION_BASIC_STATUS_ACTIVE_VISIBLE_NOT_FOCUS = 0x03, | ||
}; |
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.
This here is a dupplicate of zzz_generated/app-common/app-common/zap-generated/enums.h
-> EmberAfApplicationBasicStatus
Problem
Fix #8895
Change overview
Add initial codegen support for Cluster object
Testing
The existing unit test covers this.