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

Reply via email to