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

Ted Ross commented on DISPATCH-569:
-----------------------------------

Having the explicit list of cases in the switch statement was done deliberately 
so we would know when the API changed and that the list of events needed to be 
re-examined for needed changes in Dispatch.  Rather than add the default, we 
should add in the new events and bump the required Proton version to the 
version in which the new events are added.

> Dispatch does not compile due to new pn_event_type_t enum values.
> -----------------------------------------------------------------
>
>                 Key: DISPATCH-569
>                 URL: https://issues.apache.org/jira/browse/DISPATCH-569
>             Project: Qpid Dispatch
>          Issue Type: Bug
>    Affects Versions: 0.7.0
>            Reporter: Alan Conway
>            Assignee: Alan Conway
>             Fix For: 0.8.0
>
>
> pn_event_handler has a switch that explicitly lists every enum value of 
> pn_event_type_t even though most of the cases do nothing. This fails to 
> compile against proton master, which has some new enum values.
> Replace the empty cases with a default: case



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to