Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Review and consider replacing some data pointer/len pairs with ByteSpan and Fixe... #7458

Closed
todo bot opened this issue Jun 8, 2021 · 1 comment
Closed

Review and consider replacing some data pointer/len pairs with ByteSpan and Fixe... #7458

todo bot opened this issue Jun 8, 2021 · 1 comment
Assignees
@emargolis
Labels
todo

Comments

@todo
Copy link

todo bot commented Jun 8, 2021

connectedhomeip/src/credentials/CHIPCert.h

Lines 309 to 312 in 85eb604

// TODO: Review and consider replacing some data pointer/len pairs with ByteSpan and FixedByteSpan types.
ByteSpan mCertificate; /**< Original raw buffer data. */
ChipDN mSubjectDN; /**< Certificate Subject DN. */
ChipDN mIssuerDN; /**< Certificate Issuer DN. */

This issue was generated by todo based on a TODO comment in 85eb604. It's been assigned to @emargolis because they committed the code.
@todo todo bot added the todo label Jun 8, 2021
@todo todo bot assigned emargolis Jun 8, 2021
@emargolis emargolis mentioned this issue Jun 14, 2021
Merged
@emargolis
Copy link
Contributor

Addressed in #7599. Can be closed

@emargolis emargolis mentioned this issue Jul 21, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@emargolis emargolis

Labels
todo
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@emargolis