summaryrefslogtreecommitdiffstats
path: root/org.eclipse.jgit.pgm.test/src/org/eclipse
diff options
context:
space:
mode:
authorCarsten Hammer <carsten.hammer@t-online.de>2019-05-21 17:04:18 +0200
committerMatthias Sohn <matthias.sohn@sap.com>2019-12-14 20:42:03 +0100
commit74bc50125d3a2cb0a7eb8c32118b3f9a9530a361 (patch)
tree48e86457ef252be344ece68d9558f72fd23b911c /org.eclipse.jgit.pgm.test/src/org/eclipse
parent1b3a555f667712ece7f17f009b5444862cbac54f (diff)
downloadjgit-74bc50125d3a2cb0a7eb8c32118b3f9a9530a361.tar.gz
jgit-74bc50125d3a2cb0a7eb8c32118b3f9a9530a361.zip
Replace chain of if statements with switch
and switch over strings where possible. Sometimes if statements are chained and form a series of comparisons against constants. Using switch statements improves readability. Bug: 545856 Change-Id: Iacb78956ee5c20db4d793e6b668508ec67466606 Signed-off-by: Carsten Hammer <carsten.hammer@t-online.de> Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
Diffstat (limited to 'org.eclipse.jgit.pgm.test/src/org/eclipse')
-rw-r--r--org.eclipse.jgit.pgm.test/src/org/eclipse/jgit/lib/CLIRepositoryTestCase.java7
1 files changed, 4 insertions, 3 deletions
diff --git a/org.eclipse.jgit.pgm.test/src/org/eclipse/jgit/lib/CLIRepositoryTestCase.java b/org.eclipse.jgit.pgm.test/src/org/eclipse/jgit/lib/CLIRepositoryTestCase.java
index befb7f62ac..1d9dca73d0 100644
--- a/org.eclipse.jgit.pgm.test/src/org/eclipse/jgit/lib/CLIRepositoryTestCase.java
+++ b/org.eclipse.jgit.pgm.test/src/org/eclipse/jgit/lib/CLIRepositoryTestCase.java
@@ -190,11 +190,12 @@ public class CLIRepositoryTestCase extends LocalDiskRepositoryTestCase {
}
protected String cmdString(String... cmds) {
- if (cmds.length == 0)
+ switch (cmds.length) {
+ case 0:
return "";
- else if (cmds.length == 1)
+ case 1:
return "\"" + escapeJava(cmds[0]) + "\"";
- else {
+ default:
StringBuilder sb = new StringBuilder(cmdString(cmds[0]));
for (int i=1; i<cmds.length; i++) {
sb.append(", ");