From 1b5b1c49d74fc0f8be2f1303b1e25a197814baff Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20Nussbaumer?= <rn@google.com>
Date: Wed, 18 Jul 2012 13:34:36 +0200
Subject: [PATCH] Another small consistency fix with if branches
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

While looking at the testability of this piece of code, I found another
consistency problem. We have two if branches instead of one, with
elif's.

Signed-off-by: RenΓ© Nussbaumer <rn@google.com>
Reviewed-by: Iustin Pop <iustin@google.com>
---
 lib/server/masterd.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/server/masterd.py b/lib/server/masterd.py
index 67ea5ca7f..71ddb90b5 100644
--- a/lib/server/masterd.py
+++ b/lib/server/masterd.py
@@ -272,7 +272,7 @@ class ClientOps:
       ops = [opcodes.OpCode.LoadOpCode(state) for state in job_def]
       return queue.SubmitJob(ops)
 
-    if method == luxi.REQ_SUBMIT_MANY_JOBS:
+    elif method == luxi.REQ_SUBMIT_MANY_JOBS:
       logging.info("Received multiple jobs")
       (job_defs, ) = args
       jobs = []
-- 
GitLab