Check for write errors in standard out and exit with error

The underlying problem is that System.out is a PrintWriter and
as such it does not throw exceptions on error, but rather just
sets a flag and continues.

This changes replaces the use of System.out with a PrintWriter-like
writer that does not catch error, but instead throw them to the
caller.

Bug: 366243
Change-Id: I44405edc4416e943b87f09a0f6ed041c6c51b046

1 files changed, 4 insertions, 8 deletions

diff --git a/org.eclipse.jgit.pgm.test/src/org/eclipse/jgit/pgm/CLIGitCommand.java b/org.eclipse.jgit.pgm.test/src/org/eclipse/jgit/pgm/CLIGitCommand.java
index 28339c71e3..78e752d44e 100644
--- a/org.eclipse.jgit.pgm.test/src/org/eclipse/jgit/pgm/CLIGitCommand.java
+++ b/org.eclipse.jgit.pgm.test/src/org/eclipse/jgit/pgm/CLIGitCommand.java
@@ -42,10 +42,7 @@
  */
 package org.eclipse.jgit.pgm;
 
-import java.io.BufferedWriter;
 import java.io.ByteArrayOutputStream;
-import java.io.OutputStreamWriter;
-import java.io.PrintWriter;
 import java.util.ArrayList;
 import java.util.List;
 
@@ -84,21 +81,20 @@ public class CLIGitCommand {
 		clp.parseArgument(argv);
 
 		final TextBuiltin cmd = bean.getSubcommand();
+		ByteArrayOutputStream baos = new ByteArrayOutputStream();
+		cmd.outs = baos;
 		if (cmd.requiresRepository())
 			cmd.init(db, null);
 		else
 			cmd.init(null, null);
-		ByteArrayOutputStream baos = new ByteArrayOutputStream();
-		cmd.out = new PrintWriter(new BufferedWriter(new OutputStreamWriter(
-				baos)));
 		try {
 			cmd.execute(bean.getArguments().toArray(
 					new String[bean.getArguments().size()]));
 		} catch (Die e) {
 			return IO.readLines(e.getMessage());
 		} finally {
-			if (cmd.out != null)
-				cmd.out.flush();
+			if (cmd.outw != null)
+				cmd.outw.flush();
 		}
 		return IO.readLines(baos.toString());
 	}