-
Notifications
You must be signed in to change notification settings - Fork 167
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
Fix typos found by codespell #292
base: master
Are you sure you want to change the base?
Fix typos found by codespell #292
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And also serial numbers need to be updated here...
8283162
to
1964149
Compare
Ah, I see, this last fail in the CI was only related to separate commit itself since the CI checks commits separately. Otherwise all looks ok here. Thanks @DimitriPapadopoulos |
1964149
to
5834963
Compare
Squashed into a single commit. |
ax_cpu_freq.m4 - [ax_cpu_freq=unknow; rm -f conftest_cpufreq]
+ [ax_cpu_freq=unknown; rm -f conftest_cpufreq]
Is the string an action or the result of something? such clarification might facilitate a decision on whether or not to modify the string ax_dist_msi.m4 typo ax_prog_emacs.m4 typo ax_sys_perlsharpbang.m4 possibly change this comment line
to
|
Thank you for the review, I'll apply the suggestions next week and look into ax_cpu_freq.m4. |
It's clearly a result, that is later used to define
The misspelling has been there from the start (6c640df in 2007).
I could change exit failurefully into exit in error. However, wasn't that an intentional pun? |
|
Was the intention
|
I guess you are one of the listed authors of ax_gcc_x86_avx_xgetbv.m4. Would you have additional explanation of the code in ax_cpu_freq.m4? |
re-usable
andreusable
can be seen as correct, but:reuse
only:tesselation
andtessellation
can be seen as correct, but:tessellation
has been in much wider use.tessellation