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

Wrapping policy #52

Closed
timsgardner opened this issue Oct 10, 2014 · 1 comment
Closed

Wrapping policy #52

timsgardner opened this issue Oct 10, 2014 · 1 comment
Milestone

Comments

@timsgardner
Copy link
Contributor

Much better to have a small number of carefully-chosen features than lots of hasty ones, especially if we have all the functionality already anyway via interop. I'm nervous about the degree of wrapping we're doing. I don't think simply making all Unity methods first-class is worth the bloat to the public API, certainly not without more solid type-hinting machinery. Wrapping also causes some weird side effects, like shadowing clojure.core (happens when we wrap Mathf). On the other hand it may well be useful to bulk zap Unity utility classes into collections of (type-hinted!) functions à la carte. Manipulating vector3's, for example, should probably be easier. My instinct is that we should manually target specific pieces of functionality that need some Clojure spiffing. Then, if it is possible to do non-jankily (which it may well not be), expose defwrapper as a public macro for users to selectively wrap Unity classes.

@nasser nasser added this to the Alpha 0.1 milestone Apr 1, 2016
@nasser
Copy link
Contributor

nasser commented Apr 26, 2016

The policy in place is to wrap conservatively when a particular use case comes up over and over again (e.g. the former linear namespace). We've also done a lot of this already and folded it all into arcadia.core, so I am calling this closed.

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

No branches or pull requests

2 participants