[ 
https://issues.apache.org/jira/browse/PROTON-964?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14661903#comment-14661903
 ] 

ASF GitHub Bot commented on PROTON-964:
---------------------------------------

Github user gemmellr commented on the pull request:

    https://github.com/apache/qpid-proton/pull/48#issuecomment-128714379
  
    Hi Bozo, sorry for the delay getting back to you, I was a bit 
tunnel-visioned trying (and failing :/) to get 0.10 out the door this week.
    
    I like the recent change giving the handler a method to handle events. I'm 
not sure it really reduces (so much as perhaps moves) the casting much, but it 
at least gives the Handler interface something to do other than onUnhandled and 
also gives the Handler the decision over how to handle an Event rather than the 
other way around, which seems nicer.
    
    It would still be good for some other folks who actually wrote the 
event/reactor stuff to take a look (like @rhs and @prestona, <poke>) but master 
is open for 0.11 at this point....


> Proton-J extensible event types
> -------------------------------
>
>                 Key: PROTON-964
>                 URL: https://issues.apache.org/jira/browse/PROTON-964
>             Project: Qpid Proton
>          Issue Type: Improvement
>          Components: proton-j
>    Affects Versions: 0.10
>            Reporter: Bozo Dragojevic
>            Assignee: Bozo Dragojevic
>             Fix For: 0.11
>
>
> Event.Type is an enum which makes it impossible to extend.
> Introduce a separate interface EventType and make Event.Type implement it.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to