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

Make reflection optional #79

Closed
bennidi opened this issue Jul 28, 2014 · 3 comments
Closed

Make reflection optional #79

bennidi opened this issue Jul 28, 2014 · 3 comments
Milestone

Comments

@bennidi
Copy link
Owner

bennidi commented Jul 28, 2014

Reflection is currently used to configure and invoke handlers. In some environments like the Android Java VM reflection is limited and/or slow, hence it would help to have an alternative way of using mbassador. There is the already custom handler invocation. After a quick look, it seems like it should be enough to provide a way of declarative creation of a handler metadata object and corresponding subscription methods.

@bennidi bennidi added this to the 1.1.12 milestone Jul 28, 2014
@bwzhang2011
Copy link

bennidi, how does it going on ? it sounds more perfect idea like custom pendingQueue messages support brought in. looking foward to those in the future release.

@Maaartinus
Copy link

Maybe you could use an annotation processor to do some work in at compile time (e.g., find all subscribers in the whole class hierarchy or generate code calling them instead of using reflection). There's a Guice derivative doing exactly this (I can't recall its name).

@bennidi
Copy link
Owner Author

bennidi commented Mar 24, 2016

Not going to happen anytime soon. Android is not a main target platform of this library. Removing reflection is a bigger piece of work.

@bennidi bennidi closed this as completed Mar 24, 2016
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

3 participants