Remove USE_PARAMETER #35
Merged
kske
merged 4 commits from f/remove-use-parameter
into develop
7 months ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'f/remove-use-parameter'
Deleting a branch is permanent. It CANNOT be undone. Continue?
Use
void.class
instead ofUSE_PARAMETER.class
which was introduced for reasons no longer present.Additionally, update the accessibility of reflective event handler methods only if that's actually necessary. That way, public handlers can be can be invoked reflectively without opening the containing module to Event Bus.
f/remove-use-parameterto Remove USE_PARAMETER 7 months agothis.listener = listener;
this.method = method;
useParameter = annotation.value() == USE_PARAMETER.class;
useParameter = annotation.value() == void.class;
|| annotation.value == Void.class
That wouldn't be the default type though. Yes, using
Void
as an event type doesn't make much sense, but it is possible, which isn't the case forvoid
.a8d858e8c7
into develop 7 months agoReviewers
a8d858e8c7
.