ppkarwasz commented on code in PR #790:
URL: https://github.com/apache/commons-io/pull/790#discussion_r2393937675
##########
src/main/java/org/apache/commons/io/output/NullAppendable.java:
##########
@@ -51,6 +53,7 @@ public Appendable append(final CharSequence csq) throws
IOException {
@Override
public Appendable append(final CharSequence csq, final int start, final
int end) throws IOException {
+ IOUtils.checkFromToIndex(csq, start, end);
Review Comment:
I fixed the Javadoc in
https://github.com/apache/commons-io/pull/790/commits/d00d5effe0da276f7c0d0caa1b1cef77f6c2103d
and added some tests to assert that arguments are validated.
The current behavior aligns with `Writer.nullWriter()`, except `close()` is
a no-op in this implementation.
--
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]
