Revert AbstractStreamBuilder changes

Author: ppkarwasz

src/main/java/org/apache/commons/io/build/AbstractStreamBuilder.java

@@ -17,7 +17,6 @@
 
 package org.apache.commons.io.build;
 
-import java.io.Closeable;
 import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
@@ -124,15 +123,7 @@ public int getBufferSizeDefault() {
     }
 
     /**
-     * Gets the origin as a {@link Channel} of the given type, if supported.
-     *
-     * <p>If the origin is not already open, it will be opened using the options provided by
-     * {@link #getOpenOptions()}.</p>
-     *
-     * <p>Calling this method transfers ownership of the underlying resource to the caller.
-     * The caller is responsible for closing the returned stream once it is no longer needed.
-     * If the origin itself implements {@link Closeable}, closing the returned stream will
-     * also close the underlying origin resource.</p>
+     * Gets a Channel from the origin with OpenOption[].
      *
      * @param channelType The channel type, not null.
      * @return A channel of the specified type.
@@ -144,7 +135,6 @@ public int getBufferSizeDefault() {
      * @see #getOpenOptions()
      * @since 2.21.0
      */
-    @SuppressWarnings("resource")
     public <C extends Channel> C getChannel(final Class<C> channelType) throws IOException {
         return checkOrigin().getChannel(channelType, getOpenOptions());
     }
@@ -159,7 +149,6 @@ public <C extends Channel> C getChannel(final Class<C> channelType) throws IOExc
      * @see AbstractOrigin#getCharSequence(Charset)
      * @since 2.13.0
      */
-    @SuppressWarnings("resource")
     public CharSequence getCharSequence() throws IOException {
         return checkOrigin().getCharSequence(getCharset());
     }
@@ -191,21 +180,12 @@ public Charset getCharsetDefault() {
      * @see AbstractOrigin#getPath()
      * @since 2.18.0
      */
-    @SuppressWarnings("resource")
     public File getFile() {
         return checkOrigin().getFile();
     }
 
     /**
-     * Gets the origin as an {@link InputStream}, if supported.
-     *
-     * <p>If the origin is not already open, it will be opened using the options provided by
-     * {@link #getOpenOptions()}.</p>
-     *
-     * <p>Calling this method transfers ownership of the underlying resource to the caller.
-     * The caller is responsible for closing the returned stream once it is no longer needed.
-     * If the origin itself implements {@link Closeable}, closing the returned stream will
-     * also close the underlying origin resource.</p>
+     * Gets an InputStream from the origin with OpenOption[].
      *
      * @return An input stream
      * @throws IllegalStateException         if the {@code origin} is {@code null}.
@@ -215,7 +195,6 @@ public File getFile() {
      * @see #getOpenOptions()
      * @since 2.13.0
      */
-    @SuppressWarnings("resource")
     public InputStream getInputStream() throws IOException {
         return checkOrigin().getInputStream(getOpenOptions());
     }
@@ -230,15 +209,7 @@ public OpenOption[] getOpenOptions() {
     }
 
     /**
-     * Gets the origin as an {@link OutputStream}, if supported.
-     *
-     * <p>If the origin is not already open, it will be opened using the options provided by
-     * {@link #getOpenOptions()}.</p>
-     *
-     * <p>Calling this method transfers ownership of the underlying resource to the caller.
-     * The caller is responsible for closing the returned stream once it is no longer needed.
-     * If the origin itself implements {@link Closeable}, closing the returned stream will
-     * also close the underlying origin resource.</p>
+     * Gets an OutputStream from the origin with OpenOption[].
      *
      * @return An OutputStream
      * @throws IllegalStateException         if the {@code origin} is {@code null}.
@@ -248,7 +219,6 @@ public OpenOption[] getOpenOptions() {
      * @see #getOpenOptions()
      * @since 2.13.0
      */
-    @SuppressWarnings("resource")
     public OutputStream getOutputStream() throws IOException {
         return checkOrigin().getOutputStream(getOpenOptions());
     }
@@ -262,43 +232,25 @@ public OutputStream getOutputStream() throws IOException {
      * @see AbstractOrigin#getPath()
      * @since 2.13.0
      */
-    @SuppressWarnings("resource")
     public Path getPath() {
         return checkOrigin().getPath();
     }
 
     /**
-     * Gets the origin as an {@link OutputStream}, if supported.
-     *
-     * <p>If the origin is not already open, it will be opened using the options provided by
-     * {@link #getOpenOptions()}.</p>
-     *
-     * <p>Calling this method transfers ownership of the underlying resource to the caller.
-     * The caller is responsible for closing the returned stream once it is no longer needed.
-     * If the origin itself implements {@link Closeable}, closing the returned stream will
-     * also close the underlying origin resource.</p>
+     * Gets a RandomAccessFile from the origin.
      *
      * @return A RandomAccessFile
      * @throws IllegalStateException         if the {@code origin} is {@code null}.
      * @throws UnsupportedOperationException if the origin cannot be converted to a {@link RandomAccessFile}.
      * @throws IOException                   if an I/O error occurs.
      * @since 2.18.0
      */
-    @SuppressWarnings("resource")
     public RandomAccessFile getRandomAccessFile() throws IOException {
         return checkOrigin().getRandomAccessFile(getOpenOptions());
     }
 
     /**
-     * Gets the origin as an {@link OutputStream}, if supported.
-     *
-     * <p>If the origin is not already open, it will be opened using the options provided by
-     * {@link #getOpenOptions()}.</p>
-     *
-     * <p>Calling this method transfers ownership of the underlying resource to the caller.
-     * The caller is responsible for closing the returned stream once it is no longer needed.
-     * If the origin itself implements {@link Closeable}, closing the returned stream will
-     * also close the underlying origin resource.</p>
+     * Gets a Reader from the origin with a Charset.
      *
      * @return A Reader
      * @throws IllegalStateException         if the {@code origin} is {@code null}.
@@ -308,21 +260,12 @@ public RandomAccessFile getRandomAccessFile() throws IOException {
      * @see #getCharset()
      * @since 2.16.0
      */
-    @SuppressWarnings("resource")
     public Reader getReader() throws IOException {
         return checkOrigin().getReader(getCharset());
     }
 
     /**
-     * Gets the origin as an {@link OutputStream}, if supported.
-     *
-     * <p>If the origin is not already open, it will be opened using the options provided by
-     * {@link #getOpenOptions()}.</p>
-     *
-     * <p>Calling this method transfers ownership of the underlying resource to the caller.
-     * The caller is responsible for closing the returned stream once it is no longer needed.
-     * If the origin itself implements {@link Closeable}, closing the returned stream will
-     * also close the underlying origin resource.</p>
+     * Gets a Writer from the origin with an OpenOption[].
      *
      * @return An writer.
      * @throws IllegalStateException         if the {@code origin} is {@code null}.
@@ -332,7 +275,6 @@ public Reader getReader() throws IOException {
      * @see #getOpenOptions()
      * @since 2.13.0
      */
-    @SuppressWarnings("resource")
     public Writer getWriter() throws IOException {
         return checkOrigin().getWriter(getCharset(), getOpenOptions());
     }