-
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
Change Media::ApplicationBasic cluster to match spec #29959
Change Media::ApplicationBasic cluster to match spec #29959
Conversation
…r list as access administer
PR #29959: Size comparison from 7879111 to d73f3e2 Increases (17 builds for bl702, bl702l, cc32xx, linux, nrfconnect, psoc6, telink)
Decreases (6 builds for cc32xx, psoc6)
Full report (73 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
@@ -26,12 +26,15 @@ limitations under the License. | |||
<description>This cluster provides information about an application running on a TV or media player device which is represented as an endpoint.</description> | |||
<attribute side="server" code="0x0000" define="APPLICATION_VENDOR_NAME" type="char_string" default="" length="32" writable="false" optional="true">VendorName</attribute> | |||
<attribute side="server" code="0x0001" define="APPLICATION_VENDOR_ID" type="vendor_id" default="0x0" min="0x0000" max="0xFFFF" writable="false" optional="true">VendorID</attribute> | |||
<attribute side="server" code="0x0002" define="APPLICATION_NAME" type="char_string" length="32" writable="false" optional="false">ApplicationName</attribute> | |||
<attribute side="server" code="0x0002" define="APPLICATION_NAME" type="long_char_string" length="256" writable="false" optional="false">ApplicationName</attribute> |
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.
Hmm. Is there a spec issue tracking the fact that this has a "desc" constraint when it should hust have a normal constraint?
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.
Not that I am aware ... only saw this because I saw the scraper saying "no constraint" and reading the description said 256 .... it should actually be 256 I guess but from an SDK perspective I would prefer bytes.
Changes:
The maximum length of the ApplicationName attribute is 256 bytes of UTF-8 characters.
) ... which is actually somewhat we may not be able to fully enforce as I believe we allocate bytes not utf8-chars. But this is better than 32.