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

ASF subversion and git services commented on PROTON-2195:
---------------------------------------------------------

Commit ae06b4180e924b392e1b816f97aada8591255b5a in qpid-proton's branch 
refs/heads/master from Andrew Stitcher
[ https://gitbox.apache.org/repos/asf?p=qpid-proton.git;h=ae06b41 ]

PROTON-2195: Finalise the proactor API around pn_event_batch_t
- Move pn_event_batch_t entirely into libqpid-proton-proactor
- Make the pn_event_batch_t implementation private
- Removed pn_event_batch_connection_driver() API as it is not used at all
  and can't be used for anything.


> Tidy up/Finalise the API around proactor event batches
> ------------------------------------------------------
>
>                 Key: PROTON-2195
>                 URL: https://issues.apache.org/jira/browse/PROTON-2195
>             Project: Qpid Proton
>          Issue Type: Improvement
>          Components: proton-c
>    Affects Versions: proton-c-0.30.0
>            Reporter: Andrew Stitcher
>            Assignee: Andrew Stitcher
>            Priority: Major
>             Fix For: proton-c-0.31.0
>
>
> There are a number of issues with the {{pn_event_batch_t}} type in proton:
>  * The {{pn_event_batch_t}} type should be a type whose details are private 
> and only meaningful to the proactor because it is only the proactor API 
> batches events for serialisation. However it is defined in the event.h header 
> file and its only operation {{pn_event_batch_next()}} is actually in the 
> proton core library.
> * The type is used internally in the connection_driver API which is in the 
> proton core library. This use is unnecessary as the batches are never even 
> exposed by the connection_driver API. Indeed it's hard to see what use the 
> connection_driver could make of an event batch since it only knows about 
> events from the proton core library which only returns events and never 
> batches.
> The proactor and connection_driver APIs which are the ones affected are both 
> marked as Unsettled APIs so these changes are currently acceptable.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org

Reply via email to