ppkarwasz commented on code in PR #790:
URL: https://github.com/apache/commons-io/pull/790#discussion_r2395841788
##########
src/main/java/org/apache/commons/io/output/NullWriter.java:
##########
@@ -77,17 +79,24 @@ public Writer append(final CharSequence csq) {
}
/**
- * Does nothing, like writing to {@code /dev/null}.
+ * Does nothing except argument validation, like writing to {@code
/dev/null}.
*
- * @param csq The character sequence to write.
- * @param start The index of the first character to write.
- * @param end The index of the first character to write (exclusive).
- * @return this writer.
+ * @param csq The character sequence from which a subsequence will be
+ * appended.
+ * If {@code csq} is {@code null}, it is treated as if it were
+ * {@code "null"}.
+ * @param start The index of the first character in the subsequence.
+ * @param end The index of the character following the last character in
the
+ * subsequence.
+ * @return {@code this} instance.
+ * @throws IndexOutOfBoundsException If {@code start} or {@code end} are
negative, {@code end} is
+ * greater than {@code csq.length()}, or
{@code start} is greater
+ * than {@code end}.
* @since 2.0
*/
@Override
public Writer append(final CharSequence csq, final int start, final int
end) {
- //to /dev/null
Review Comment:
Fixed in
https://github.com/apache/commons-io/pull/790/commits/b80971821c227b09de4e2aec5a2adcee7b4b4a3b
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
