-
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
Cleanup IM #12606
Cleanup IM #12606
Conversation
80972ca
to
88e811f
Compare
PR #12606: Size comparison from b1e0023 to 88e811f Decreases (32 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Full report (39 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
88e811f
to
74970da
Compare
PR #12606: Size comparison from bfdbd45 to 74970da Decreases (32 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Full report (39 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.
Lots of trivial changes, to other reviewers: check "Hide whitespace" for better review experience.
74970da
to
608655d
Compare
PR #12606: Size comparison from f350919 to 608655d Decreases (20 builds for efr32, k32w, linux, p6, qpg, telink)
Full report (22 builds for efr32, k32w, linux, p6, qpg, telink)
|
--Move ConstructCommandPath function into CommandPathIB, update all usage across the codes. --Use reference delcartion for usages on all Create* functions from IM MessageDef, update all usage across the code. --Update the missing error-check for IM messageDef.
608655d
to
dd9f581
Compare
PR #12606: Size comparison from f350919 to dd9f581 Decreases (32 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Full report (39 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Problem
--Move ConstructCommandPath function from Command class into CommandPathIB, update all usages across the codes.
--Use reference declaration for all uages on Create* functions from IM MessageDef, update all usages across the codes.
--Update the missing error-check for IM messageDef.
Change overview
See above
Testing
The existing test covers.