Skip to content

Spelling - #207

Closed
jsoref wants to merge 14 commits into
pbatard:masterfrom
jsoref:spelling
Closed

Spelling#207
jsoref wants to merge 14 commits into
pbatard:masterfrom
jsoref:spelling

Conversation

@jsoref

@jsoref jsoref commented Jan 14, 2021

Copy link
Copy Markdown
Contributor

This PR corrects misspellings identified by the check-spelling action.

The misspellings have been reported at jsoref@8edd7ec#commitcomment-45944093

The action reports that the changes in this PR would make it happy: jsoref@bdacd93

Note: this PR does not include the action. If you're interested in running a spell check on every PR and push, that can be offered separately.

jsoref added 14 commits January 13, 2021 21:47
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

@jsoref jsoref left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrections generally generated by Google Sheets.

Note that the action in this specific instance generated a slightly buggy comment -- I'm currently working on improving the action. But as noted, the action isn't included in the PR, and I wouldn't include a prerelease in a general PR.

Comment thread configure.ac
Comment thread libwdi/msapi_utf8.h
#define isspaceU(c) isspace((unsigned char)(c))
#define isxdigitU(c) isxdigit((unsigned char)(c))
// NB: other issomething() calls are not implemented as they may require multibyte UTF-8 sequences to be converted
// NB: other is*() calls are not implemented as they may require multibyte UTF-8 sequences to be converted

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this isn't a specific thing, this seems like a reasonable correction.

Fwiw, this is the only correction I made. All other corrections were suggested by Google Sheets's spell checker.

@pbatard

pbatard commented Feb 9, 2021

Copy link
Copy Markdown
Owner

Thanks.

@pbatard pbatard closed this in cc0b96e Feb 9, 2021
@jsoref
jsoref deleted the spelling branch February 9, 2021 17:51
NikhilNarayana pushed a commit to project-slippi/libwdi that referenced this pull request Oct 2, 2021
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.

2 participants