Better Javadoc

Author: garydgregory

src/changes/changes.xml

@@ -71,6 +71,7 @@ The <action> type attribute can be add,update,fix,remove.
       <action dev="ggregory" type="fix" issue="IO-873" due-to="Gary Gregory">java.lang.ArithmeticException: long overflow java.lang.Math.addExact(Math.java:932) at org.apache.commons.io.file.attribute.FileTimes.ntfsTimeToDate(long).</action>
       <action dev="ggregory" type="fix"                due-to="Gary Gregory">FileTimes.toNtfsTime(*) methods can overflow result values.</action>
       <action dev="ggregory" type="fix"                due-to="Gary Gregory">Fix Javadoc for ChunkedOutputStream.Builder.</action>
+      <action dev="ggregory" type="fix"                due-to="Gary Gregory">General Javadoc improvements.</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/input/BoundedInputStream.java

@@ -28,7 +28,7 @@
 /**
  * Reads bytes up to a maximum count and stops once reached.
  * <p>
- * To build an instance, see {@link AbstractBuilder}.
+ * To build an instance: Use the {@link #builder()} to access all features.
  * </p>
  * <p>
  * By default, a {@link BoundedInputStream} is <em>unbound</em>; so make sure to call {@link AbstractBuilder#setMaxCount(long)}.
@@ -145,7 +145,7 @@ public T setCount(final long count) {
          * Default is {@value IOUtils#EOF}, negative means unbound.
          * </p>
          *
-         * @param maxCount The maximum number of bytes to return.
+         * @param maxCount The maximum number of bytes to return, negative means unbound.
          * @return {@code this} instance.
          */
         public T setMaxCount(final long maxCount) {
@@ -321,6 +321,9 @@ public static Builder builder() {
 
     /**
      * Constructs a new {@link BoundedInputStream} that wraps the given input stream and is <em>unbounded</em>.
+     * <p>
+     * To build an instance: Use the {@link #builder()} to access all features.
+     * </p>
      *
      * @param in The wrapped input stream.
      * @deprecated Use {@link AbstractBuilder#get()}.
@@ -342,7 +345,7 @@ public BoundedInputStream(final InputStream in) {
      * Constructs a new {@link BoundedInputStream} that wraps the given input stream and limits it to a certain size.
      *
      * @param inputStream The wrapped input stream.
-     * @param maxCount    The maximum number of bytes to return.
+     * @param maxCount    The maximum number of bytes to return, negative means unbound.
      * @deprecated Use {@link AbstractBuilder#get()}.
      */
     @Deprecated