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.

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

camera/component information: URI zero termination #1807

Merged
merged 2 commits into from
Mar 2, 2022

Conversation

julianoes
Copy link
Collaborator

This way it's much easier to deal with these message as the last char can just be set to '\0' before handing it. Otherwise the string needs to be copied first.

This way it's much easier to deal with this message as the last char can
just be set to '\0' before handing it. Otherwise it needs to be copied
first.
This way it's much easier to deal with this message as the last char can
just be set to '\0' before handing it. Otherwise it needs to be copied
first.
@hamishwillee
Copy link
Collaborator

@julianoes Would there be any implications to existing implementations? My thought was no, but want to make sure before I merge.

@hamishwillee
Copy link
Collaborator

Merging. As discussed in dev call, only implication is if a particular camera uses the whole URL, and that is unlikely. Otherwise better for all.

@hamishwillee hamishwillee merged commit 5186324 into master Mar 2, 2022
@hamishwillee hamishwillee deleted the pr-uri-zero-termination branch March 2, 2022 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants