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

Classes should use a three letter prefix. #16

Closed
ianegordon opened this issue Feb 2, 2017 · 2 comments
Closed

Classes should use a three letter prefix. #16

ianegordon opened this issue Feb 2, 2017 · 2 comments

Comments

@ianegordon
Copy link
Contributor

"Two-letter prefixes... are reserved by Apple for use in framework classes."
https://developer.apple.com/library/content/documentation/Cocoa/Conceptual/ProgrammingWithObjectiveC/Conventions/Conventions.html

Consider changing LA -> LOT to avoid namespace collisions.
@hansemannn
Copy link
Contributor

Btw, LA is already used by Apple for the LocalAuthentification (LA) framework. Could be a search-and-replace thing, but also a breaking change that requires a version bump. Maybe schedule as part of a 2.0.0 release?

@buba447
Copy link
Collaborator

buba447 commented Feb 5, 2017

Ohh good to know!

@ianegordon ianegordon mentioned this issue Feb 7, 2017
Closed
@buba447 buba447 closed this as completed Feb 7, 2017
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

No branches or pull requests
3 participants
@ianegordon @buba447 @hansemannn