-
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
Fix Darwin framework APIs to accept arbitrary types for command invokes #11767
Fix Darwin framework APIs to accept arbitrary types for command invokes #11767
Conversation
eb4c9ef
to
a93814f
Compare
PR #11767: Size comparison from 507ba15 to a93814f Increases (1 build for linux)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
a93814f
to
dd93835
Compare
PR #11767: Size comparison from 507ba15 to dd93835 Increases (1 build for linux)
Full report (21 builds for efr32, k32w, linux, p6, qpg, telink)
|
dd93835
to
e70639f
Compare
PR #11767: Size comparison from 507ba15 to e70639f Increases (1 build for linux)
Full report (34 builds for esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
e70639f
to
4a49b4e
Compare
PR #11767: Size comparison from 507ba15 to 4a49b4e Increases (1 build for linux)
Full report (38 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.
Thanks !
This also extends the return value handling, and the test comparison bits, to all types.
4a49b4e
to
45f60f1
Compare
45f60f1
to
4d9313a
Compare
Fast tracking, given this is a platform maintainer updating their platform - and also adding tests. |
PR #11767: Size comparison from 7c47bed to 4d9313a Increases (1 build for linux)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
…es (project-chip#11767) * Generate Objective-C types to represent Matter structs and command payloads. * Improve Objective-C to C++ value conversion to handle more cases. * Change Darwin cluster command invocations to take payload structs. * Update the Darwin tests to use the new command APIs. * Update return value decoding for commands on Darwin to use the new APIs. This also extends the return value handling, and the test comparison bits, to all types. * Enable a bunch of complex-types tests on Darwin.
Review note: It may be easier to review the commits separately.
Problem
Lists, structs, nullables, optional command fields all broken.
Change overview
Fix all that stuff.
Testing
Enables various complex-types command tests on Darwin.