This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-io.git


The following commit(s) were added to refs/heads/master by this push:
     new e20545c  Javadoc.
e20545c is described below

commit e20545c7724b3202d3dbc95b89adcfdc49552baa
Author: Gary Gregory <gardgreg...@gmail.com>
AuthorDate: Fri Aug 9 11:13:58 2019 -0400

    Javadoc.
---
 .../apache/commons/io/output/AppendableWriter.java | 107 +++++++++++----------
 1 file changed, 55 insertions(+), 52 deletions(-)

diff --git a/src/main/java/org/apache/commons/io/output/AppendableWriter.java 
b/src/main/java/org/apache/commons/io/output/AppendableWriter.java
index 93b2d4f..d0ec457 100644
--- a/src/main/java/org/apache/commons/io/output/AppendableWriter.java
+++ b/src/main/java/org/apache/commons/io/output/AppendableWriter.java
@@ -26,6 +26,7 @@ import java.util.Objects;
  * <p>
  * For example, can be used with a {@link java.lang.StringBuilder}
  * or {@link java.lang.StringBuffer}.
+ * </p>
  *
  * @since 2.7
  * @see java.lang.Appendable
@@ -37,7 +38,7 @@ public class AppendableWriter <T extends Appendable> extends 
Writer {
     private final T appendable;
 
     /**
-     * Construct a new instance with the specified appendable.
+     * Constructs a new instance with the specified appendable.
      *
      * @param appendable the appendable to write to
      */
@@ -46,49 +47,16 @@ public class AppendableWriter <T extends Appendable> 
extends Writer {
     }
 
     /**
-     * Write a character to the underlying appendable.
-     *
-     * @param c the character to write
-     * @throws IOException upon error
-     */
-    @Override
-    public void write(final int c) throws IOException {
-        appendable.append((char)c);
-    }
-
-    /**
-     * Writes a portion of an array of characters to the underlying appendable.
-     *
-     * @param cbuf an array with the characters to write
-     * @param off offset from which to start writing characters
-     * @param len number of characters to write
-     * @throws IOException upon error
-     */
-    @Override
-    public void write(final char[] cbuf, final int off, final int len) throws 
IOException {
-        Objects.requireNonNull(cbuf, "Character array is missing");
-        if (len < 0 || (off + len) > cbuf.length) {
-            throw new IndexOutOfBoundsException("Array Size=" + cbuf.length +
-                    ", offset=" + off + ", length=" + len);
-        }
-        for (int i = 0; i < len; i++) {
-            appendable.append(cbuf[off + i]);
-        }
-    }
-
-    /**
-     * Writes a portion of a String to the underlying appendable.
+     * Appends the specified character to the underlying appendable.
      *
-     * @param str a string
-     * @param off offset from which to start writing characters
-     * @param len number of characters to write
+     * @param c the character to append
+     * @return this writer
      * @throws IOException upon error
      */
     @Override
-    public void write(final String str, final int off, final int len) throws 
IOException {
-        // appendable.append will add "null" for a null String; add an 
explicit null check
-        Objects.requireNonNull(str, "String is missing");
-        appendable.append(str, off, off + len);
+    public Writer append(final char c) throws IOException {
+        appendable.append(c);
+        return this;
     }
 
     /**
@@ -120,16 +88,13 @@ public class AppendableWriter <T extends Appendable> 
extends Writer {
     }
 
     /**
-     * Appends the specified character to the underlying appendable.
+     * Closes the stream. This implementation does nothing.
      *
-     * @param c the character to append
-     * @return this writer
      * @throws IOException upon error
      */
     @Override
-    public Writer append(final char c) throws IOException {
-        appendable.append(c);
-        return this;
+    public void close() throws IOException {
+        // noop
     }
 
     /**
@@ -139,24 +104,62 @@ public class AppendableWriter <T extends Appendable> 
extends Writer {
      */
     @Override
     public void flush() throws IOException {
+        // noop
     }
 
     /**
-     * Closes the stream. This implementation does nothing.
+     * Return the target appendable.
+     *
+     * @return the target appendable
+     */
+    public T getAppendable() {
+        return appendable;
+    }
+
+    /**
+     * Writes a portion of an array of characters to the underlying appendable.
      *
+     * @param cbuf an array with the characters to write
+     * @param off offset from which to start writing characters
+     * @param len number of characters to write
      * @throws IOException upon error
      */
     @Override
-    public void close() throws IOException {
+    public void write(final char[] cbuf, final int off, final int len) throws 
IOException {
+        Objects.requireNonNull(cbuf, "Character array is missing");
+        if (len < 0 || (off + len) > cbuf.length) {
+            throw new IndexOutOfBoundsException("Array Size=" + cbuf.length +
+                    ", offset=" + off + ", length=" + len);
+        }
+        for (int i = 0; i < len; i++) {
+            appendable.append(cbuf[off + i]);
+        }
     }
 
     /**
-     * Return the target appendable.
+     * Writes a character to the underlying appendable.
      *
-     * @return the target appendable
+     * @param c the character to write
+     * @throws IOException upon error
      */
-    public T getAppendable() {
-        return appendable;
+    @Override
+    public void write(final int c) throws IOException {
+        appendable.append((char)c);
+    }
+
+    /**
+     * Writes a portion of a String to the underlying appendable.
+     *
+     * @param str a string
+     * @param off offset from which to start writing characters
+     * @param len number of characters to write
+     * @throws IOException upon error
+     */
+    @Override
+    public void write(final String str, final int off, final int len) throws 
IOException {
+        // appendable.append will add "null" for a null String; add an 
explicit null check
+        Objects.requireNonNull(str, "String is missing");
+        appendable.append(str, off, off + len);
     }
 
 }

Reply via email to