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

Codespell errors #67

Closed
AnuyaWelling2801 opened this issue Nov 7, 2022 · 2 comments · Fixed by #70
Closed

Codespell errors #67

AnuyaWelling2801 opened this issue Nov 7, 2022 · 2 comments · Fixed by #70

Comments

@AnuyaWelling2801
Copy link

Codespell errors on the following lines -
src/ittnotify/ittnotify_static.h:353: hist ==> heist, his
src/ittnotify/ittnotify_static.h:353: hist ==> heist, his
include/ittnotify.h:3649: immidiatelly ==> immediately
include/ittnotify.h:3962: hist ==> heist, his
include/ittnotify.h:3967: hist ==> heist, his
include/ittnotify.h:3972: hist ==> heist, his
include/ittnotify.h:3976: hist ==> heist, his
abrown added a commit to abrown/ittapi that referenced this issue Nov 9, 2022
@abrown
Fix spelling of doc comment
cb209bb 
This misspelling was found by @AnuyaWelling2801 in intel#67; closes intel#67.
@abrown abrown mentioned this issue Nov 9, 2022
Merged
abrown added a commit to abrown/ittapi that referenced this issue Nov 9, 2022
@abrown
Fix spelling of doc comment
58c9acd 
This misspelling was found by @AnuyaWelling2801 in intel#67; closes intel#67.
@abrown
Copy link
Contributor

abrown commented Nov 9, 2022

I think the use of hist is not an error but an abbreviation of "histogram." The other error should be fixed in #70.

@abrown
Copy link
Contributor

abrown commented Nov 9, 2022

@AnuyaWelling2801, do you want to give a +1 over there?

abrown added a commit that referenced this issue Nov 10, 2022
@abrown
Fix spelling of doc comment (#70)
f04184c 
This misspelling was found by @AnuyaWelling2801 in #67; closes #67.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix spelling of doc comment abrown/ittapi
2 participants
@abrown @AnuyaWelling2801