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

Update documentation of utf8 #1205

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update documentation of utf8 #1205

wants to merge 2 commits into from

Conversation

rmbeer
Copy link
Contributor

@rmbeer rmbeer commented Dec 12, 2020

Correction of the 3 lines in the utf8.txt documentation, which better explains what happens when the original string is released.

@rmbeer
Copy link
Contributor Author

rmbeer commented Dec 12, 2020

Fallo manejando github.

@rmbeer rmbeer reopened this Dec 12, 2020
@pedro-w
Copy link
Contributor

pedro-w commented Dec 12, 2020

You can link a PR to the issue it's going to fix - see GitHub docs

@SiegeLord
Copy link
Member

That's not quite the issue with the documentation. The issue is that the returned string is also tied to the info struct, not the other way around. The sentence should be "The returned string is valid while the underlying memory buffer and the info struct are valid."

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.

3 participants