-
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 ListBuilder/Parser and StructBuilder/Parser #11231
Add ListBuilder/Parser and StructBuilder/Parser #11231
Conversation
PR #11231: Size comparison from 345723d to bb87c8e Increases (2 builds for p6, telink)
Full report (2 builds for p6, telink)
|
bb87c8e
to
99a9c43
Compare
PR #11231: Size comparison from 345723d to 99a9c43 Increases (26 builds for efr32, esp32, k32w, linux, nrfconnect, p6, qpg, telink)
Decreases (4 builds for efr32, mbed)
Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
99a9c43
to
8f67ca3
Compare
8f67ca3
to
f16ff33
Compare
PR #11231: Size comparison from 345723d to f16ff33 Increases (26 builds for efr32, esp32, k32w, linux, nrfconnect, p6, qpg, telink)
Decreases (4 builds for efr32, mbed)
Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, 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.
This is fine as far as it goes, but again I think a setup more like the cluster-object setup might be less code...
--Fix ListBuilder naming, which should be ArrayBuilder --Add ListBuilder/Parser and StructBuilder/Praser, the enablement for them would be follow-up PR
f16ff33
to
364526f
Compare
PR #11231: Size comparison from bf585f8 to 364526f Increases (26 builds for efr32, esp32, k32w, linux, nrfconnect, p6, qpg, telink)
Decreases (3 builds for mbed)
Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
PR #11231: Size comparison from bf585f8 to be80d63 Increases (26 builds for efr32, esp32, k32w, linux, nrfconnect, p6, qpg, telink)
Decreases (3 builds for mbed)
Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
1 day fast track: straight forward change that has unit tests, created by domain owner and with separate approval of another domain owner. |
c3de94e
to
34eec02
Compare
34eec02
to
1163da6
Compare
PR #11231: Size comparison from 1073bb2 to 1163da6 Increases (26 builds for efr32, esp32, k32w, linux, nrfconnect, p6, qpg, telink)
Decreases (4 builds for efr32, mbed)
Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
* Add ListBuilder/Parser and StructBuilder/Parser --Fix ListBuilder naming, which should be ArrayBuilder --Add ListBuilder/Parser and StructBuilder/Praser, the enablement for them would be follow-up PR * Fix java codegen missing in master * address comments
Problem
--Fix ListBuilder naming, which should be ArrayBuilder
--Refactor common List/Struct logic from IM IB/Messages to ListBuilder/Parser and StructBuilder/Parser, update some of messageDef and others would be updated in the following week.
--List/Array/Struct are base module for all IB/Messages for IM, and create the dedicate builder/parser would reduce lots of duplicate code.
Change overview
See above
Testing
The existing tests are using them.
Add unit tests