From: Gerhard Sittig <redacted>
Date: Sun, 26 Feb 2017 18:12:05 +0000 (+0100)
Subject: runtc: fixup coding style (conditionals, braces)
X-Git-Url: https://sigrok.org/gitaction?a=commitdiff_plain;h=59060ffcaada7201ad6942ce2c9bb67361ccbddd;p=sigrok-test.git

runtc: fixup coding style (conditionals, braces)

When a conditional has one complex branch (multiple instructions), both
branches are supposed to have braces around them. Instead of putting
extra braces around single instructions, eliminate the "else" branch
since the "if" branch already ends in break or return respectively.
---

diff --git a/decoder/runtc.c b/decoder/runtc.c
index c7462b4..4b534a2 100644
--- a/decoder/runtc.c
+++ b/decoder/runtc.c
@@ -459,16 +459,16 @@ static int run_testcase(const char *infile, GSList *pdlist, struct output *op)
 			if (!strcmp(decoder_class[0], op->class)) {
 				op->class_idx = idx;
 				break;
-			} else
-				idx++;
+			}
+			idx++;
 			l = l->next;
 		}
 		if (op->class_idx == -1) {
 			ERR("Output class '%s' not found in decoder %s.",
 					op->class, pd->name);
 			return FALSE;
-		} else
-			DBG("Class %s index is %d", op->class, op->class_idx);
+		}
+		DBG("Class %s index is %d", op->class, op->class_idx);
 	}
 
 	sr_session_start(sr_sess);