From: Matthias Sohn <matthias.sohn@sap.com>
Date: Sun, 20 Jan 2019 00:30:24 +0000 (+0100)
Subject: pgm: Fix missing braces in Commit.run()
X-Git-Tag: v5.3.0.201903061415-rc1~1^2~117
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=refs%2Fchanges%2F86%2F135386%2F1;p=jgit.git

pgm: Fix missing braces in Commit.run()

Change-Id: Ia9e7e846ba1abfdb490896e5bcb82e2c0039439c
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
---

diff --git a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Commit.java b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Commit.java
index ebbb932ac4..d1ab529deb 100644
--- a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Commit.java
+++ b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Commit.java
@@ -89,10 +89,12 @@ class Commit extends TextBuiltin {
 	protected void run() {
 		try (Git git = new Git(db)) {
 			CommitCommand commitCmd = git.commit();
-			if (author != null)
+			if (author != null) {
 				commitCmd.setAuthor(RawParseUtils.parsePersonIdent(author));
-			if (message != null)
+			}
+			if (message != null) {
 				commitCmd.setMessage(message);
+			}
 			if (noGpgSign) {
 				commitCmd.setSign(Boolean.FALSE);
 			} else if (gpgSigningKey != null) {
@@ -101,13 +103,17 @@ class Commit extends TextBuiltin {
 					commitCmd.setSigningKey(gpgSigningKey);
 				}
 			}
-			if (only && paths.isEmpty())
+			if (only && paths.isEmpty()) {
 				throw die(CLIText.get().pathsRequired);
-			if (only && all)
+			}
+			if (only && all) {
 				throw die(CLIText.get().onlyOneOfIncludeOnlyAllInteractiveCanBeUsed);
-			if (!paths.isEmpty())
-				for (String p : paths)
+			}
+			if (!paths.isEmpty()) {
+				for (String p : paths) {
 					commitCmd.setOnly(p);
+				}
+			}
 			commitCmd.setAmend(amend);
 			commitCmd.setAll(all);
 			Ref head = db.exactRef(Constants.HEAD);
@@ -122,12 +128,13 @@ class Commit extends TextBuiltin {
 			}
 
 			String branchName;
-			if (!head.isSymbolic())
+			if (!head.isSymbolic()) {
 				branchName = CLIText.get().branchDetachedHEAD;
-			else {
+			} else {
 				branchName = head.getTarget().getName();
-				if (branchName.startsWith(Constants.R_HEADS))
+				if (branchName.startsWith(Constants.R_HEADS)) {
 					branchName = branchName.substring(Constants.R_HEADS.length());
+				}
 			}
 			outw.println("[" + branchName + " " + commit.name() + "] " //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
 					+ commit.getShortMessage());