Skip to content

JANITORIAL: Fix spelling mistakes in comments and text #7593

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

Closed
wants to merge 14 commits into from
Closed

JANITORIAL: Fix spelling mistakes in comments and text #7593

wants to merge 14 commits into from

Conversation

raziel-
Copy link
Contributor

@raziel- raziel- commented May 16, 2024

No code or variables were touched, these are solely typo fixes in either text or comments.

"seeked" should be "sought", but i was told on another project that it is meant as a follow up and explanation to the command of "seek"ing in data, i think it would be best to distinguish it with an added apostrophe (seek'ed).
That way it won' t come up as error anymore and the meaning is still preserved.

I hope i didn't break any rules by simply adding a PR here

@ocornut ocornut added the doc label May 16, 2024
ocornut pushed a commit that referenced this pull request May 16, 2024
@ocornut
Copy link
Owner

ocornut commented May 16, 2024

I merged this, thank you!
However, it is possible that the 1 typo in stb_truetype.h and stb_textedit.h may be overridden by a future update as they are updated from another repository.

@ocornut ocornut closed this May 16, 2024
@raziel- raziel- deleted the patch-1 branch May 16, 2024 14:22
@raziel-
Copy link
Contributor Author

raziel- commented May 16, 2024

where does upstream come from for the two files?

@ocornut
Copy link
Owner

ocornut commented May 16, 2024

https://github.com/nothings/stb/ but good luck getting typo fixes merged in (they are 8 open typos PR).

@raziel-
Copy link
Contributor Author

raziel- commented May 16, 2024

ugh...132 open PRs...I better skip that then...thanks for the heads up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants