-
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
[Scenes] AttributeValue refactor #33260
[Scenes] AttributeValue refactor #33260
Conversation
3158f7d
to
bc0e0a9
Compare
PR #33260: Size comparison from bf73003 to bc0e0a9 Increases above 0.2%:
Increases (22 builds for cc13x4_26x4, cyw30739, efr32, esp32, linux, nrfconnect, psoc6, stm32, telink)
Decreases (6 builds for efr32, linux, psoc6)
Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
bc0e0a9
to
9c2af2c
Compare
PR #33260: Size comparison from 8edcecd to 9c2af2c Full report (19 builds for bl602, bl702, bl702l, cc32xx, mbed, nxp, qpg)
|
9c2af2c
to
89a938b
Compare
PR #33260: Size comparison from 8edcecd to 89a938b Increases above 0.2%:
Increases (25 builds for cc13x4_26x4, cyw30739, efr32, esp32, linux, nrfconnect, psoc6, stm32, telink)
Decreases (6 builds for efr32, linux, psoc6)
Full report (80 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
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.
The crash issues should get fixed here, the rest in followups.
…of colorcontrol in scenes
Co-authored-by: Boris Zbarsky <[email protected]>
7ee42b1
to
0e1f0b4
Compare
PR #33260: Size comparison from d454ffb to 0e1f0b4 Increases above 0.2%:
Full report (49 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, tizen)
|
PR #33260: Size comparison from d454ffb to 69a5fc1 Increases above 0.2%:
Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
* Refactor AttributeValue to the appropriate Value* in AttributeValueList * Updated attribute value in test * Applied comments about typing and commented code, as well as example of colorcontrol in scenes * Made static check on same type instead of size * Update src/app/zap-templates/zcl/data-model/chip/scene.xml Co-authored-by: Boris Zbarsky <[email protected]> * Added checks for missing values in level control and on off * Added checks for HasValue in color control apply scene handler * Added check on out of range values in the level control handler * Updated Zap generated files --------- Co-authored-by: Boris Zbarsky <[email protected]>
Updated scene.xml AttributeValuePairStruct to match the spec.
Updated scene handlers and test scripts to use the new valueUnsigned* instead of AttributeValue.