connectedhomeip icon indicating copy to clipboard operation
connectedhomeip copied to clipboard

Investigate better TLV API to SDK consumer

Open yunhanw-google opened this issue 3 years ago • 3 comments

I am very worried about exposing the raw TLV to SDK consumers. The TLV reader API is not particularly easy to use, unfortunately. Again, I guess we can do this for the moment, but it seems to me that we're eventually going to want a nicer API. What the size cost of that would be is a hard question, but I would really like something safer to use than "here, read some tlv; good luck", especially when the rules around reading TLV are not well-internalized even by CHIP developers and are not well-described in the spec.

https://github.com/project-chip/connectedhomeip/pull/5198#pullrequestreview-614895525

yunhanw-google avatar Mar 18 '21 02:03 yunhanw-google

The real question is whether "here's the TLV for the event list" is the right API at all to expose as an external SDK API.

bzbarsky-apple avatar Mar 18 '21 15:03 bzbarsky-apple

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] avatar Jul 25 '22 04:07 stale[bot]

This stale issue has been automatically closed. Thank you for your contributions.

stale[bot] avatar Oct 18 '22 19:10 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] avatar Apr 19 '23 14:04 stale[bot]