Deprecate IOUtils.readFully(InputStream, int) in favor of

toByteArray(InputStream, int)

Author: garydgregory

src/changes/changes.xml

@@ -52,6 +52,7 @@ The <action> type attribute can be add,update,fix,remove.
       <action type="fix" dev="ggregory"                due-to="Stanislav Fort, Gary Gregory">ValidatingObjectInputStream does not validate dynamic proxy interfaces.</action>
       <action type="fix" dev="pkarwasz"                due-to="Piotr P. Karwasz">BoundedInputStream.getRemaining() now reports Long.MAX_VALUE instead of 0 when no limit is set.</action>
       <action type="fix" dev="pkarwasz"                due-to="Piotr P. Karwasz">BoundedInputStream.available() correctly accounts for the maximum read limit.</action>
+      <action type="fix" dev="ggregory"                due-to="Gary Gregory, Piotr P. Karwasz">Deprecate IOUtils.readFully(InputStream, int) in favor of toByteArray(InputStream, int).</action>
       <!-- ADD -->
       <action dev="ggregory" type="add"                due-to="strangelookingnerd, Gary Gregory">FileUtils#byteCountToDisplaySize() supports Zettabyte, Yottabyte, Ronnabyte and Quettabyte #763.</action>
       <action dev="ggregory" type="add"                due-to="strangelookingnerd, Gary Gregory">Add org.apache.commons.io.FileUtils.ONE_RB #763.</action>

src/main/java/org/apache/commons/io/IOUtils.java

@@ -2030,11 +2030,11 @@ public static void readFully(final InputStream input, final byte[] buffer, final
      * @throws IllegalArgumentException if length is negative.
      * @throws EOFException             if the number of bytes read was incorrect.
      * @since 2.5
+     * @deprecated Use {@link #toByteArray(InputStream, int)}.
      */
+    @Deprecated
     public static byte[] readFully(final InputStream input, final int length) throws IOException {
-        final byte[] buffer = byteArray(length);
-        readFully(input, buffer, 0, buffer.length);
-        return buffer;
+        return toByteArray(input, length);
     }
 
     /**