[GitHub] qpid-dispatch pull request #382: DISPATCH-1129 - Moved call to forwarding mu...

2018-09-25 Thread ganeshmurthy 
Github user ganeshmurthy closed the pull request at:

https://github.com/apache/qpid-dispatch/pull/382


---

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

[GitHub] qpid-dispatch pull request #382: DISPATCH-1129 - Moved call to forwarding mu...

2018-09-24 Thread ted-ross 
Github user ted-ross commented on a diff in the pull request:

https://github.com/apache/qpid-dispatch/pull/382#discussion_r219978301
  
--- Diff: src/router_core/router_core_private.h ---
@@ -440,6 +440,17 @@ struct qdr_link_ref_t {
 ALLOC_DECLARE(qdr_link_ref_t);
 DEQ_DECLARE(qdr_link_ref_t, qdr_link_ref_list_t);
 
+
+typedef struct qdr_forward_deliver_info_t {
--- End diff --

This is only used in forwarder.c.  Why not move all of this into that file?


---

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

[GitHub] qpid-dispatch pull request #382: DISPATCH-1129 - Moved call to forwarding mu...

2018-09-24 Thread ganeshmurthy 
GitHub user ganeshmurthy opened a pull request:

https://github.com/apache/qpid-dispatch/pull/382

DISPATCH-1129 - Moved call to forwarding multicast messages to end of…

… forwarder function. Also modified qd_message_check_LH to not validate 
the message if bufferes have already been freed

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ganeshmurthy/qpid-dispatch AARON-PROBLEM

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/qpid-dispatch/pull/382.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #382


commit 00edc49fec82aaabcf4d3bfc65e24ed0b73d1d62
Author: Ganesh Murthy 
Date:   2018-09-24T17:06:47Z

DISPATCH-1129 - Moved call to forwarding multicast messages to end of 
forwarder function. Also modified qd_message_check_LH to not validate the 
message if bufferes have already been freed




---

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