diff options
| author | David Pursehouse <david.pursehouse@sonymobile.com> | 2012-10-23 18:09:58 +0900 |
|---|---|---|
| committer | David Pursehouse <david.pursehouse@sonymobile.com> | 2012-10-23 18:10:15 +0900 |
| commit | d309677f9dd631b6a956659dd45ef91e4944656d (patch) | |
| tree | d13ce1cfafd587db051a822f72feaa088c5241af /pygerrit/stream.py | |
| parent | f6ffad3f6f924813a410b684e3af1bfdaad80344 (diff) | |
| download | pygerrit-d309677f9dd631b6a956659dd45ef91e4944656d.tar.gz | |
Reduce nested `if` blocks in stream handling
Change-Id: I283773fb6b68da1878ede9001d18b5b7d0aff569
Diffstat (limited to 'pygerrit/stream.py')
| -rw-r--r-- | pygerrit/stream.py | 19 |
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) |
