tSimplify following on from prior dead code removal - electrum - Electrum Bitcoin wallet HTML git clone https://git.parazyd.org/electrum DIR Log DIR Files DIR Refs DIR Submodules --- DIR commit fb5e8068ceb64cbcebd0618b945c8f2371956896 DIR parent f9547a194e620458b17e8fee3ca12ca038d2baf5 HTML Author: Neil Booth <kyuupichan@gmail.com> Date: Sat, 12 Sep 2015 11:25:14 +0900 Simplify following on from prior dead code removal Diffstat: M lib/network.py | 39 +++++++++++--------------------- 1 file changed, 13 insertions(+), 26 deletions(-) --- DIR diff --git a/lib/network.py b/lib/network.py t@@ -534,8 +534,14 @@ class Network(util.DaemonThread): self.pending_sends.append((messages, callback)) def process_pending_sends(self): - sends = self.pending_sends - self.pending_sends = [] + # Requests needs connectivity. If we don't have an interface, + # we cannot process them. + if not self.interface: + return + + with self.lock: + sends = self.pending_sends + self.pending_sends = [] for messages, callback in sends: subs = filter(lambda (m,v): m.endswith('.subscribe'), messages) t@@ -544,30 +550,11 @@ class Network(util.DaemonThread): if sub not in self.subscriptions[callback]: self.subscriptions[callback].append(sub) - unsent = [] - for message in messages: - if not self.process_request(message, callback): - unsent.append(message) - - if unsent: - with self.lock: - self.pending_sends.append((unsent, callback)) - - # FIXME: inline this function - def process_request(self, request, callback): - '''Returns true if the request was processed.''' - method, params = request - - # This request needs connectivity. If we don't have an - # interface, we cannot process it. - if not self.interface: - return False - - if self.debug: - self.print_error("-->", request) - message_id = self.queue_request(method, params) - self.unanswered_requests[message_id] = method, params, callback - return True + for method, params in messages: + if self.debug: + self.print_error("-->", request) + message_id = self.queue_request(method, params) + self.unanswered_requests[message_id] = method, params, callback def connection_down(self, server): '''A connection to server either went down, or was never made.