Remove redundancy in EventBus classes

Open magicprinc opened this issue 2 years ago • 0 comments

Redundant fields were removed, but other fixes were forgotten

× MessageConsumerImpl.registered → use isRegistered()

× dispatch(Message,ContextInternal,Handler<Message<T>>) → dispatch(Message,ContextInternal)

× EventBusImpl.addLocalRegistration: don't create ConcurrentCyclicSequence every call: CHM merge → compute

PR to remember situation: https://github.com/eclipse-vertx/vert.x/pull/4859

Issue: https://github.com/eclipse-vertx/vert.x/issues/4848