summaryrefslogtreecommitdiff
path: root/pygerrit
diff options
context:
space:
mode:
authorDavid Pursehouse <david.pursehouse@sonymobile.com>2012-10-23 18:09:58 +0900
committerDavid Pursehouse <david.pursehouse@sonymobile.com>2012-10-23 18:10:15 +0900
commitd309677f9dd631b6a956659dd45ef91e4944656d (patch)
treed13ce1cfafd587db051a822f72feaa088c5241af /pygerrit
parentf6ffad3f6f924813a410b684e3af1bfdaad80344 (diff)
downloadpygerrit-d309677f9dd631b6a956659dd45ef91e4944656d.tar.gz
Reduce nested `if` blocks in stream handling
Change-Id: I283773fb6b68da1878ede9001d18b5b7d0aff569
Diffstat (limited to 'pygerrit')
-rw-r--r--pygerrit/stream.py19
1 files changed, 9 insertions, 10 deletions
diff --git a/pygerrit/stream.py b/pygerrit/stream.py
index cc1247b..ed09e73 100644
--- a/pygerrit/stream.py
+++ b/pygerrit/stream.py
@@ -83,13 +83,12 @@ class GerritStream(Thread):
while not self._stop.is_set():
data = poller.poll()
for (handle, event) in data:
- if handle == stdout.channel.fileno():
- if event == POLLIN:
- try:
- line = stdout.readline()
- json_data = json.loads(line)
- self._gerrit.put_event(json_data)
- except (ValueError, IOError), err:
- self._error_event(err)
- except GerritError, err:
- logging.error("Failed to put event: %s", err)
+ if handle == stdout.channel.fileno() and event == POLLIN:
+ try:
+ line = stdout.readline()
+ json_data = json.loads(line)
+ self._gerrit.put_event(json_data)
+ except (ValueError, IOError), err:
+ self._error_event(err)
+ except GerritError, err:
+ logging.error("Failed to put event: %s", err)