Author: bayard
Date: Tue Mar  7 23:17:00 2006
New Revision: 384140

URL: http://svn.apache.org/viewcvs?rev=384140&view=rev
Log:
Applying patch #17677. Remove the Writer API in favour of the PrintWriter API 
it is using elsewhere anyway - with the advantage that the IOException throws 
go away

Modified:
    
jakarta/commons/proper/cli/trunk/src/java/org/apache/commons/cli2/util/HelpFormatter.java
    
jakarta/commons/proper/cli/trunk/src/test/org/apache/commons/cli2/util/HelpFormatterTest.java

Modified: 
jakarta/commons/proper/cli/trunk/src/java/org/apache/commons/cli2/util/HelpFormatter.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/cli/trunk/src/java/org/apache/commons/cli2/util/HelpFormatter.java?rev=384140&r1=384139&r2=384140&view=diff
==============================================================================
--- 
jakarta/commons/proper/cli/trunk/src/java/org/apache/commons/cli2/util/HelpFormatter.java
 (original)
+++ 
jakarta/commons/proper/cli/trunk/src/java/org/apache/commons/cli2/util/HelpFormatter.java
 Tue Mar  7 23:17:00 2006
@@ -15,9 +15,7 @@
  */
 package org.apache.commons.cli2.util;
 
-import java.io.IOException;
 import java.io.PrintWriter;
-import java.io.Writer;
 
 import java.util.ArrayList;
 import java.util.Collections;
@@ -156,10 +154,8 @@
 
     /**
      * Prints the Option help.
-     * @throws IOException if an error occurs
      */
-    public void print()
-        throws IOException {
+    public void print() {
         printHeader();
         printException();
         printUsage();
@@ -170,10 +166,8 @@
 
     /**
      * Prints any error message.
-     * @throws IOException if an error occurs
      */
-    public void printException()
-        throws IOException {
+    public void printException() {
         if (exception != null) {
             printDivider();
             printWrapped(exception.getMessage());
@@ -182,10 +176,8 @@
 
     /**
      * Prints detailed help per option.
-     * @throws IOException if an error occurs
      */
-    public void printHelp()
-        throws IOException {
+    public void printHelp() {
         printDivider();
 
         final Option option;
@@ -253,10 +245,8 @@
 
     /**
      * Prints a single line of usage information (wrapping if necessary)
-     * @throws IOException if an error occurs
      */
-    public void printUsage()
-        throws IOException {
+    public void printUsage() {
         printDivider();
 
         final StringBuffer buffer = new StringBuffer("Usage:\n");
@@ -267,10 +257,8 @@
 
     /**
      * Prints a header string if necessary
-     * @throws IOException if an error occurs
      */
-    public void printHeader()
-        throws IOException {
+    public void printHeader() {
         if (header != null) {
             printDivider();
             printWrapped(header);
@@ -279,10 +267,8 @@
 
     /**
      * Prints a footer string if necessary
-     * @throws IOException if an error occurs
      */
-    public void printFooter()
-        throws IOException {
+    public void printFooter() {
         if (footer != null) {
             printWrapped(footer);
             printDivider();
@@ -292,10 +278,8 @@
     /**
      * Prints a string wrapped if necessary
      * @param text the string to wrap
-     * @throws IOException if an error occurs
      */
-    protected void printWrapped(final String text)
-        throws IOException {
+    protected void printWrapped(final String text) {
         for (final Iterator i = wrap(text, pageWidth).iterator(); 
i.hasNext();) {
             printGutterLeft();
             pad((String) i.next(), pageWidth, out);
@@ -333,8 +317,7 @@
 
     protected static void pad(final String text,
                               final int width,
-                              final Writer writer)
-        throws IOException {
+                              final PrintWriter writer) {
         final int left;
 
         // write the text and record how many characters written

Modified: 
jakarta/commons/proper/cli/trunk/src/test/org/apache/commons/cli2/util/HelpFormatterTest.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/cli/trunk/src/test/org/apache/commons/cli2/util/HelpFormatterTest.java?rev=384140&r1=384139&r2=384140&view=diff
==============================================================================
--- 
jakarta/commons/proper/cli/trunk/src/test/org/apache/commons/cli2/util/HelpFormatterTest.java
 (original)
+++ 
jakarta/commons/proper/cli/trunk/src/test/org/apache/commons/cli2/util/HelpFormatterTest.java
 Tue Mar  7 23:17:00 2006
@@ -414,28 +414,28 @@
     public void testPad()
         throws IOException {
         final StringWriter writer = new StringWriter();
-        HelpFormatter.pad("hello", 10, writer);
+        HelpFormatter.pad("hello", 10, new PrintWriter(writer));
         assertEquals("hello     ", writer.toString());
     }
 
     public void testPad_Null()
         throws IOException {
         final StringWriter writer = new StringWriter();
-        HelpFormatter.pad(null, 10, writer);
+        HelpFormatter.pad(null, 10, new PrintWriter(writer));
         assertEquals("          ", writer.toString());
     }
 
     public void testPad_TooLong()
         throws IOException {
         final StringWriter writer = new StringWriter();
-        HelpFormatter.pad("hello world", 10, writer);
+        HelpFormatter.pad("hello world", 10, new PrintWriter(writer));
         assertEquals("hello world", writer.toString());
     }
 
     public void testPad_TooShort()
         throws IOException {
         final StringWriter writer = new StringWriter();
-        HelpFormatter.pad("hello world", -5, writer);
+        HelpFormatter.pad("hello world", -5, new PrintWriter(writer));
         assertEquals("hello world", writer.toString());
     }
 



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to