-
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
Allow piecemeal migration from EmberAf non-enum-class enums to cluster-objects enum classes #11900
Allow piecemeal migration from EmberAf non-enum-class enums to cluster-objects enum classes #11900
Conversation
PR #11900: Size comparison from 2b52b36 to 5476e3f Full report (13 builds for efr32, k32w, p6, qpg, telink)
|
5476e3f
to
d7dcfa6
Compare
PR #11900: Size comparison from 2b52b36 to d7dcfa6 Decreases (2 builds for linux)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
d7dcfa6
to
04f9472
Compare
PR #11900: Size comparison from aa8a6d0 to 04f9472 Decreases (1 build for linux)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
c93ae1e
to
f141678
Compare
PR #11900: Size comparison from b83502b to f141678 Decreases (1 build for linux)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
f141678
to
4606be8
Compare
PR #11900: Size comparison from 2e85d48 to 4606be8 Decreases (2 builds for linux)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Introduces an explicit list of spec enums that are generated as not enum class. We can then drive this list down to 0.
4606be8
to
1063c5a
Compare
PR #11900: Size comparison from 7d55d2e to 1063c5a Decreases (1 build for linux)
Full report (10 builds for linux, p6)
|
1063c5a
to
80e1fa6
Compare
PR #11900: Size comparison from 2bd6bd6 to 80e1fa6 Decreases (1 build for linux)
Full report (17 builds for esp32, linux, mbed, p6)
|
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.
That sounds pretty reasonable. I like the patch that allows what is currently in use, but do not let us introduce new weakly typed enum.
80e1fa6
to
b1f2245
Compare
b1f2245
to
0880b32
Compare
PR #11900: Size comparison from 4bac98f to 0880b32 Decreases (1 build for linux)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Problem
No real plausible path to migrate to strongly typed cluster-objects enums.
Change overview
Introduce such a path, so it can be done on a per-enum basis, not for everything at once (which seems hard to imagine happening).
Testing
Tree compiles. No expected behavior changes.