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

Change the DebugInterface, PPCDebugInterface, and DSPDebugInterface to use CamelCase function names. #128

Merged

Conversation

lioncash
Copy link
Member

@lioncash lioncash commented Mar 3, 2014

This is the standard coding convention in the codebase, so our interfaces should use it too.

…o use CamelCase names.

This is the standard coding convention in the codebase, so our interfaces should use it too.
@BhaaLseN
Copy link
Member

BhaaLseN commented Mar 3, 2014

lgtm, altho i probably have tro break it to you that most commonly "camel case" refers to "lower camel case" (first letter in lower case, which is what it was before). I hate that convention myself (and always assumed pascalCase was that one; with the p stretching downwards, while CamelCase had two humps upwards just like the camel) - but apparently the almighty wikipedia tells me that the correct term here is "PascalCase" or "Upper CamelCase" =/

@delroth
Copy link
Member

delroth commented Mar 3, 2014

LGTM.

delroth added a commit that referenced this pull request Mar 3, 2014
…names

Change the DebugInterface, PPCDebugInterface, and DSPDebugInterface to use CamelCase function names.
@delroth delroth merged commit 91676ad into dolphin-emu:master Mar 3, 2014
@lioncash
Copy link
Member Author

lioncash commented Mar 3, 2014

@BhaaLseN Noted. Will keep it in mind if I ever make a change like this again :)

@lioncash lioncash deleted the normalize-debug-interface-func-names branch March 3, 2014 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants