Avoid unnecessary boxing and unboxing of int and long values

- Avoid unnecessary boxing and unboxing of long values in
FileUtils.sizeOf(File)
- Avoid unnecessary boxing and unboxing of int values in
UncheckedBufferedReader.read()
- Avoid unnecessary boxing and unboxing of int values in
UncheckedFilterInputStream.available() and read()
- Avoid unnecessary boxing and unboxing of int values in
UncheckedFilterReader.read()

Author: garydgregory

src/changes/changes.xml

@@ -58,6 +58,10 @@ The <action> type attribute can be add,update,fix,remove.
       <action dev="ggregory" type="fix"                due-to="Gary Gregory">The Consumer to IOUtils.closeQuietly(Closeable, Consumer) now accepts Exception, not just IOException.</action>
       <action dev="ggregory" type="fix"                due-to="Gary Gregory">The Consumer to IOUtils.close(Closeable, IOConsumer) now accepts wrapped Exception, not just IOException.</action>
       <action dev="ggregory" type="fix"                due-to="Gary Gregory">Use Uncheck.getAsBoolean(IOBooleanSupplier) to avoid boxing and unboxing of boolean values.</action>
+      <action dev="ggregory" type="fix"                due-to="Gary Gregory">Avoid unnecessary boxing and unboxing of long values in FileUtils.sizeOf(File).</action>
+      <action dev="ggregory" type="fix"                due-to="Gary Gregory">Avoid unnecessary boxing and unboxing of int values in UncheckedBufferedReader.read().</action>
+      <action dev="ggregory" type="fix"                due-to="Gary Gregory">Avoid unnecessary boxing and unboxing of int values in UncheckedFilterInputStream.available() and read().</action>
+      <action dev="ggregory" type="fix"                due-to="Gary Gregory">Avoid unnecessary boxing and unboxing of int values in UncheckedFilterReader.read().</action>
       <!-- ADD -->
       <action dev="ggregory" type="add" issue="IO-860" due-to="Nico Strecker, Gary Gregory">Add ThrottledInputStream.Builder.setMaxBytes(long, ChronoUnit).</action>
       <action dev="ggregory" type="add"                due-to="Gary Gregory">Add IOIterable.</action>

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

@@ -2939,7 +2939,7 @@ private static boolean setTimes(final File sourceFile, final File targetFile) {
      * @since 2.0
      */
     public static long sizeOf(final File file) {
-        return Uncheck.get(() -> PathUtils.sizeOf(file.toPath()));
+        return Uncheck.getAsLong(() -> PathUtils.sizeOf(file.toPath()));
     }
 
     /**
@@ -2985,7 +2985,7 @@ public static long sizeOfDirectory(final File directory) {
         } catch (final FileNotFoundException e) {
             throw new UncheckedIOException(e);
         }
-        return Uncheck.get(() -> PathUtils.sizeOfDirectory(directory.toPath()));
+        return Uncheck.getAsLong(() -> PathUtils.sizeOfDirectory(directory.toPath()));
     }
 
     /**

src/main/java/org/apache/commons/io/input/UncheckedBufferedReader.java

@@ -148,7 +148,7 @@ public void mark(final int readAheadLimit) throws UncheckedIOException {
      */
     @Override
     public int read() throws UncheckedIOException {
-        return Uncheck.get(super::read);
+        return Uncheck.getAsInt(super::read);
     }
 
     /**

src/main/java/org/apache/commons/io/input/UncheckedFilterInputStream.java

@@ -123,7 +123,7 @@ private UncheckedFilterInputStream(final InputStream inputStream) {
      */
     @Override
     public int available() throws UncheckedIOException {
-        return Uncheck.get(super::available);
+        return Uncheck.getAsInt(super::available);
     }
 
     /**
@@ -139,7 +139,7 @@ public void close() throws UncheckedIOException {
      */
     @Override
     public int read() throws UncheckedIOException {
-        return Uncheck.get(super::read);
+        return Uncheck.getAsInt(super::read);
     }
 
     /**

src/main/java/org/apache/commons/io/input/UncheckedFilterReader.java

@@ -140,7 +140,7 @@ public void mark(final int readAheadLimit) throws UncheckedIOException {
      */
     @Override
     public int read() throws UncheckedIOException {
-        return Uncheck.get(super::read);
+        return Uncheck.getAsInt(super::read);
     }
 
     /**

src/test/java/org/apache/commons/io/file/FilesUncheckTest.java

@@ -279,7 +279,7 @@ public void testNewBufferedWriterPathOpenOptionArray() {
 
     @Test
     public void testNewByteChannelPathOpenOptionArray() {
-        assertEquals(0, Uncheck.get(() -> {
+        assertEquals(0, Uncheck.getAsLong(() -> {
             try (SeekableByteChannel c = FilesUncheck.newByteChannel(FILE_PATH_EMPTY, StandardOpenOption.READ)) {
                 return c.size();
             }
@@ -290,7 +290,7 @@ public void testNewByteChannelPathOpenOptionArray() {
     public void testNewByteChannelPathSetOfQextendsOpenOptionFileAttributeOfQArray() {
         final Set<OpenOption> options = new HashSet<>();
         options.add(StandardOpenOption.READ);
-        assertEquals(0, Uncheck.get(() -> {
+        assertEquals(0, Uncheck.getAsLong(() -> {
             try (SeekableByteChannel c = FilesUncheck.newByteChannel(FILE_PATH_EMPTY, options, EMPTY_FILE_ATTRIBUTES_ARRAY)) {
                 return c.size();
             }
@@ -326,7 +326,7 @@ public void testNewDirectoryStreamPathString() {
 
     @Test
     public void testNewInputStream() {
-        assertEquals(0, Uncheck.get(() -> {
+        assertEquals(0, Uncheck.getAsInt(() -> {
             try (InputStream in = FilesUncheck.newInputStream(FILE_PATH_EMPTY, StandardOpenOption.READ)) {
                 return in.available();
             }