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

Support bracket syntax for Objective-C autolinks #362

Closed
pcantrell opened this issue Nov 13, 2015 · 5 comments
Closed

Support bracket syntax for Objective-C autolinks #362

pcantrell opened this issue Nov 13, 2015 · 5 comments
Assignees

Comments

@pcantrell
Copy link
Collaborator

Autolinking should support:

  • [RLMRealm defaultRealm]
  • +[RLMRealm defaultRealm]
  • -[RLMRealm writeCopyToPath:encryptionKey:error:]

Notes:

  • Ensure that RLMRealm.configuration still works after this change.
  • Not necessary at this time to prevent dot-separated references from working, even if they’re not valid Obj-C.
@pcantrell pcantrell self-assigned this Nov 13, 2015
@jpsim
Copy link
Collaborator

jpsim commented Nov 13, 2015

Not necessary at this time to prevent dot-separated references from working, even if they’re not valid Obj-C.

RLMRealm.configuration isn't valid ObjC but is certainly a lot nicer to write than -[RLMRealm configuration], and the former to me implies a property, while the latter is more of an instance method, which has some semantic difference.

@pigeondotdev
Copy link
Contributor

Any progress on this?

@jpsim
Copy link
Collaborator

jpsim commented Nov 26, 2016

None to report.

@1ec5
Copy link
Collaborator

1ec5 commented Dec 18, 2016

#707 implements the +[RLMRealm defaultRealm] and -[RLMRealm writeCopyToPath:encryptionKey:error:] syntaxes without regressing the RLMRealm.configuration syntax.

@1ec5
Copy link
Collaborator

1ec5 commented Dec 21, 2016

More syntaxes we should autolink:

  • +defaultRealm
  • -writeCopyToPath:encryptionKey:error:
  • +[NSValue(MGLAdditions) valueWithMGLLineJoin:]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants