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

Fix some misspells #8264

Merged
merged 2 commits into from May 20, 2020
Merged

Fix some misspells #8264

merged 2 commits into from May 20, 2020

Conversation

sampletext32
Copy link
Contributor

Note: in main.cpp there are many dirs similar to Program Files, so tip should be appropriate.

Note: in main.cpp there are many dirs similar to Program Files, so tip should be appropriate.
@@ -233,7 +233,7 @@
<addaction name="separator"/>
<addaction name="actionManage_Users"/>
<addaction name="confSavedataManagerAct"/>
<addaction name="actionManage_RAP_Licences"/>
<addaction name="actionManage_RAP_Licenses"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

  • "License is both a noun and a verb in the United States."
  • "If you live in any other English-speaking country, you will spell it licence when you use it as a noun and license when you use it as a verb."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

https://github.com/RPCS3/rpcs3/blob/master/rpcs3/rpcs3qt/main_window.cpp#L279
This line says about RAP licenSe file and it's definitely not a verb.

https://github.com/RPCS3/rpcs3/blob/7a7ac625cd35c3d59f1d109aa791f2964073c2b0/rpcs3/Emu/Cell/Modules/cellSysutilMisc.cpp
This one contains definition, I believe, directly from hw. USA-like.

Also every GNU headers in files are licenSe. (USA-like)
Main RPCS3 licenSe file is at last USA-like.

So I assume we all use USA-like english and Licenses is correct here

IMHO, it's much easier to use one word instead of two, because it's less confusing and doesn't catch the eye of somebody like me.

@Nekotekina Nekotekina merged commit 1a8fb61 into RPCS3:master May 20, 2020
@sampletext32 sampletext32 deleted the fix-misspells branch May 20, 2020 22:31
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.

None yet

3 participants