Javadoc

Sort members

Author: garydgregory

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

@@ -87,6 +87,8 @@ public Builder setCharSequence(final CharSequence source) {
 
     }
 
+    private static final int NO_MARK = -1;
+
     /**
      * Constructs a new {@link Builder}.
      *
@@ -97,8 +99,6 @@ public static Builder builder() {
         return new Builder();
     }
 
-    private static final int NO_MARK = -1;
-
     private final CharsetEncoder charsetEncoder;
     private final CharBuffer cBuf;
     private final ByteBuffer bBuf;

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

@@ -46,10 +46,11 @@
  * To build an instance, see {@link Builder}.
  * </p>
  * <pre>{@code
- * BufferedInputStream s = new BufferedInputStream(new GzipInputStream(MemoryMappedFileInputStream.builder()
- *   .setPath(path)
- *   .setBufferSize(256 * 1024)
- *   .get()));}
+ * BufferedInputStream s = new BufferedInputStream(new GzipInputStream(
+ *   MemoryMappedFileInputStream.builder()
+ *     .setPath(path)
+ *     .setBufferSize(256 * 1024)
+ *     .get()));}
  * </pre>
  * <p>
  * should outperform:
@@ -58,10 +59,11 @@
  * new GzipInputStream(new MemoryMappedFileInputStream(path))
  * </pre>
  * <pre>{@code
- * GzipInputStream s = new GzipInputStream(MemoryMappedFileInputStream.builder()
- *   .setPath(path)
- *   .setBufferSize(256 * 1024)
- *   .get());}
+ * GzipInputStream s = new GzipInputStream(
+ *   MemoryMappedFileInputStream.builder()
+ *     .setPath(path)
+ *     .setBufferSize(256 * 1024)
+ *     .get());}
  * </pre>
  *
  * @since 2.12.0

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

@@ -55,7 +55,10 @@
  * InputStream inputStream = ...
  * Charset cs = ...
  * InputStreamReader reader = new InputStreamReader(inputStream, cs);
- * ReaderInputStream in2 = new ReaderInputStream(reader, cs);
+ * ReaderInputStream in2 = ReaderInputStream.builder()
+ *   .setReader(reader)
+ *   .setCharset(cs)
+ *   .get();
  * </pre>
  * <p>
  * {@link ReaderInputStream} implements the same transformation as {@link java.io.OutputStreamWriter}, except that the control flow is reversed: both classes

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

@@ -39,7 +39,10 @@
  * </p>
  *
  * <pre>
- * UnsynchronizedBufferedInputStream buf = new UnsynchronizedBufferedInputStream(new FileInputStream(&quot;file.java&quot;));
+ * UnsynchronizedBufferedInputStream s = new UnsynchronizedBufferedInputStream.Builder().
+ *   .setInputStream(new FileInputStream(&quot;file.java&quot;))
+ *   .setBufferSize(8192)
+ *   .get();
  * </pre>
  * <p>
  * Provenance: Apache Harmony and modified.

src/main/java/org/apache/commons/io/output/WriterOutputStream.java

@@ -54,7 +54,10 @@
  * OutputStream out = ...
  * Charset cs = ...
  * OutputStreamWriter writer = new OutputStreamWriter(out, cs);
- * WriterOutputStream out2 = new WriterOutputStream(writer, cs);
+ * WriterOutputStream out2 = WriterOutputStream.builder()
+ *   .setWriter(writer)
+ *   .setCharset(cs)
+ *   .get();
  * </pre>
  * <p>
  * {@link WriterOutputStream} implements the same transformation as {@link java.io.InputStreamReader}, except that the control flow is reversed: both classes

src/test/java/org/apache/commons/io/FileUtilsTest.java

@@ -1182,14 +1182,6 @@ public void testCopyFileToOutputStream() throws Exception {
         assertArrayEquals(expected, destination.toByteArray(), "Check Full copy");
     }
 
-    @Test
-    public void testCreateParentDirectories() throws IOException {
-        // If a directory already exists, nothing happens.
-        FileUtils.createParentDirectories(FileUtils.current());
-        // null is a noop
-        FileUtils.createParentDirectories(null);
-    }
-
     @Test
     public void testCopyToDirectoryWithDirectory() throws IOException {
         final File destDirectory = new File(tempDirFile, "destination");
@@ -1357,6 +1349,14 @@ public void testCountFolders2FileSize4() {
         assertEquals(8, FileUtils.sizeOfDirectory(Paths.get("src/test/resources/org/apache/commons/io/dirs-2-file-size-4").toFile()));
     }
 
+    @Test
+    public void testCreateParentDirectories() throws IOException {
+        // If a directory already exists, nothing happens.
+        FileUtils.createParentDirectories(FileUtils.current());
+        // null is a noop
+        FileUtils.createParentDirectories(null);
+    }
+
     @Test
     public void testDecodeUrl() {
         assertEquals("", FileUtils.decodeUrl(""));

src/test/java/org/apache/commons/io/IOUtilsTest.java

@@ -111,6 +111,13 @@ public class IOUtilsTest {
      * methods are called. (JT)
      */
 
+    @BeforeAll
+    @AfterAll
+    public static void beforeAll() {
+        // Not required, just to exercise the method and make sure there are no adverse side-effect when recycling thread locals.
+        IO.clear();
+    }
+
     @TempDir
     public File temporaryFolder;
 
@@ -130,13 +137,6 @@ private void assertEqualContent(final byte[] b0, final byte[] b1) {
         assertArrayEquals(b0, b1, "Content not equal according to java.util.Arrays#equals()");
     }
 
-    @BeforeAll
-    @AfterAll
-    public static void beforeAll() {
-        // Not required, just to exercise the method and make sure there are no adverse side-effect when recycling thread locals.
-        IO.clear();
-    }
-
     @BeforeEach
     public void setUp() {
         try {