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

Allow Subscription without Annotations #108

Closed
dcowden opened this issue Apr 11, 2015 · 3 comments
Closed

Allow Subscription without Annotations #108

dcowden opened this issue Apr 11, 2015 · 3 comments

Comments

@dcowden
Copy link

dcowden commented Apr 11, 2015

We have a project where we would love to use MBassador. In our use case, though, we'd prefer to avoid the use of annotations to mark hander methods, preferring instead the more 'old-fashioned way' where we have listeners that implement a particular interface.

More specifically, we have a class like this

public interface OurSpecialListener{
public void onEvent(Event e );
}

and then classes implement this interface in order to listen.

Is there a way to use MBassador in this situation? For a greenfield project, we agree that the annotations way is cleaner, and we're completely sold that the inner workings of MBassador is awesome, so we'd like to get the performance benefits, but stay compatible with our current listener structures

@dcowden
Copy link
Author

dcowden commented Apr 11, 2015

Surfing through other issues, it appears that #67 might solve this need:

If so, let me know and I will close thise one.

@bennidi
Copy link
Owner

bennidi commented Apr 12, 2015

It's true. #67 would provide you with a convenient solution. Unfortunately, I currently lack the time to implement this enhancement. Maybe there is someone who wants to pick up that issue? Most of the work would have to be done within the MetadataReader. It shouldn't be too hard to do but a bit of pain since the Java reflection API really is far from convenient.

@dcowden
Copy link
Author

dcowden commented Apr 12, 2015

Ok Closing this one, #67 captures the need.

@dcowden dcowden closed this as completed Apr 12, 2015
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

2 participants