You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by gm...@apache.org on 2017/04/25 13:10:15 UTC

qpid-dispatch git commit: DISPATCH-749 - unmap_all_addresses unmaps just half of addresses

Repository: qpid-dispatch
Updated Branches:
  refs/heads/master 2c820bb19 -> 8f92b2029


DISPATCH-749 - unmap_all_addresses unmaps just half of addresses

iterating over a list being updated needs deep copy of the list

Signed-off-by: Pavel Moravec <pm...@redhat.com>

This closes #160


Project: http://git-wip-us.apache.org/repos/asf/qpid-dispatch/repo
Commit: http://git-wip-us.apache.org/repos/asf/qpid-dispatch/commit/8f92b202
Tree: http://git-wip-us.apache.org/repos/asf/qpid-dispatch/tree/8f92b202
Diff: http://git-wip-us.apache.org/repos/asf/qpid-dispatch/diff/8f92b202

Branch: refs/heads/master
Commit: 8f92b20291ebab1aab291cdba95c2b15c184ca12
Parents: 2c820bb
Author: Pavel Moravec <pm...@redhat.com>
Authored: Sat Apr 22 16:44:53 2017 +0200
Committer: Ganesh Murthy <gm...@redhat.com>
Committed: Tue Apr 25 09:09:43 2017 -0400

----------------------------------------------------------------------
 python/qpid_dispatch_internal/router/node.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/qpid-dispatch/blob/8f92b202/python/qpid_dispatch_internal/router/node.py
----------------------------------------------------------------------
diff --git a/python/qpid_dispatch_internal/router/node.py b/python/qpid_dispatch_internal/router/node.py
index b6c26cb..448e684 100644
--- a/python/qpid_dispatch_internal/router/node.py
+++ b/python/qpid_dispatch_internal/router/node.py
@@ -533,8 +533,8 @@ class RouterNode(object):
 
     def unmap_all_addresses(self):
         self.mobile_address_sequence = 0
-        for addr in self.mobile_addresses:
-            self.unmap_address(addr)
+        while self.mobile_addresses:
+            self.unmap_address(self.mobile_addresses[0])
 
 
     def overwrite_addresses(self, addrs):


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org