Francesco Romani has uploaded a new change for review. Change subject: monitor: simplify should_run implementation ......................................................................
monitor: simplify should_run implementation Make the method a couple of lines shorter. Change-Id: Id41a43f430d09dfba7eded27f58f4036dcfa48a6 Signed-off-by: Francesco Romani <from...@redhat.com> --- M mom/Monitor.py 1 file changed, 2 insertions(+), 4 deletions(-) git pull ssh://gerrit.ovirt.org:29418/mom refs/changes/78/41378/1 diff --git a/mom/Monitor.py b/mom/Monitor.py index 3b59afa..5ff5742 100644 --- a/mom/Monitor.py +++ b/mom/Monitor.py @@ -190,7 +190,5 @@ """ Helper to determine if the Monitor should continue to run. """ - if self.config.getint('__int__', 'running') == 1 and not self._terminate: - return True - else: - return False + return (self.config.getint('__int__', 'running') == 1 and + not self._terminate) -- To view, visit https://gerrit.ovirt.org/41378 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Id41a43f430d09dfba7eded27f58f4036dcfa48a6 Gerrit-PatchSet: 1 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Francesco Romani <from...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches