diff options
author | Robin Rosenberg <robin.rosenberg@dewire.com> | 2012-09-22 16:40:16 +0200 |
---|---|---|
committer | Robin Rosenberg <robin.rosenberg@dewire.com> | 2012-09-23 23:00:35 +0200 |
commit | 7de1154d21427340387237f5d45dc9af79712915 (patch) | |
tree | eafb920466d3ba404ae6810b353c1b50a473b83d /org.eclipse.jgit.pgm/src | |
parent | 78c865942849de420592bb822e9e1fd85cc95ade (diff) | |
download | jgit-7de1154d21427340387237f5d45dc9af79712915.tar.gz jgit-7de1154d21427340387237f5d45dc9af79712915.zip |
Remove uses of TextBuiltin.out in favor of outw
These came from patches in review in parallel with the introduction
of the exception throwing print writer.
Change-Id: I1c27fa276eb1fcf12ad19792049c35cb52518c16
Diffstat (limited to 'org.eclipse.jgit.pgm/src')
3 files changed, 18 insertions, 14 deletions
diff --git a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Merge.java b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Merge.java index eee98d409d..8c769e6076 100644 --- a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Merge.java +++ b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Merge.java @@ -87,7 +87,7 @@ class Merge extends TextBuiltin { switch (result.getMergeStatus()) { case ALREADY_UP_TO_DATE: - out.println(CLIText.get().alreadyUpToDate); + outw.println(CLIText.get().alreadyUpToDate); break; case FAST_FORWARD: outw.println(result.getMergeStatus().toString()); diff --git a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Push.java b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Push.java index 2c10cf98f5..534e6de250 100644 --- a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Push.java +++ b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Push.java @@ -252,7 +252,7 @@ class Push extends TextBuiltin { private void printUpdateLine(final char flag, final String summary, final String srcRef, final String destRef, final String message) throws IOException { - out.format(" %c %-17s", valueOf(flag), summary); + outw.format(" %c %-17s", valueOf(flag), summary); if (srcRef != null) outw.format(" %s ->", abbreviateRef(srcRef, true)); diff --git a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Status.java b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Status.java index 64e38599b5..6abbfb0354 100644 --- a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Status.java +++ b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Status.java @@ -43,6 +43,7 @@ package org.eclipse.jgit.pgm; +import java.io.IOException; import java.text.MessageFormat; import java.util.ArrayList; import java.util.Collection; @@ -69,10 +70,10 @@ class Status extends TextBuiltin { boolean firstHeader = true; if (head != null && head.isSymbolic()) { String branch = Repository.shortenRefName(head.getLeaf().getName()); - out.println(CLIText.formatLine( + outw.println(CLIText.formatLine( MessageFormat.format(CLIText.get().onBranch, branch))); } else - out.println(CLIText.formatLine(CLIText.get().notOnAnyBranch)); + outw.println(CLIText.formatLine(CLIText.get().notOnAnyBranch)); // List changes org.eclipse.jgit.api.Status status = new Git(db).status().call(); Collection<String> added = status.getAdded(); @@ -122,22 +123,24 @@ class Status extends TextBuiltin { } } - protected void printSectionHeader(String pattern, Object... arguments) { - out.println(CLIText.formatLine(MessageFormat.format(pattern, arguments))); + protected void printSectionHeader(String pattern, Object... arguments) + throws IOException { + outw.println(CLIText.formatLine(MessageFormat + .format(pattern, arguments))); if (!pattern.equals("")) - out.println(CLIText.formatLine("")); - out.flush(); + outw.println(CLIText.formatLine("")); + outw.flush(); } - protected int printList(Collection<String> list) { + protected int printList(Collection<String> list) throws IOException { if (!list.isEmpty()) { List<String> sortedList = new ArrayList<String>(list); java.util.Collections.sort(sortedList); for (String filename : sortedList) { - out.println(CLIText.formatLine(String.format( + outw.println(CLIText.formatLine(String.format( statusFileListFormat, filename))); } - out.flush(); + outw.flush(); return list.size(); } else return 0; @@ -146,7 +149,8 @@ class Status extends TextBuiltin { protected int printList(String status1, String status2, String status3, Collection<String> list, Collection<String> set1, Collection<String> set2, - @SuppressWarnings("unused") Collection<String> set3) { + @SuppressWarnings("unused") Collection<String> set3) + throws IOException { List<String> sortedList = new ArrayList<String>(list); java.util.Collections.sort(sortedList); for (String filename : sortedList) { @@ -158,9 +162,9 @@ class Status extends TextBuiltin { else // if (set3.contains(filename)) prefix = status3; - out.println(CLIText.formatLine(String.format( + outw.println(CLIText.formatLine(String.format( statusFileListFormatWithPrefix, prefix, filename))); - out.flush(); + outw.flush(); } return list.size(); } |