garydgregory commented on code in PR #784:
URL: https://github.com/apache/commons-io/pull/784#discussion_r2366415963
##########
src/main/java/org/apache/commons/io/build/AbstractOrigin.java:
##########
@@ -227,7 +476,7 @@ public CharSequence getCharSequence(final Charset charset) {
@Override
public InputStream getInputStream(final OpenOption... options) throws
IOException {
// TODO Pass in a Charset? Consider if call sites actually need
this.
- return
CharSequenceInputStream.builder().setCharSequence(getCharSequence(Charset.defaultCharset())).get();
+ return
CharSequenceInputStream.builder().setCharSequence(getCharSequence(null)).get();
Review Comment:
How is this change related to Channels? This is a giant PR already. Can it
be focused on just the new feature? Or is this a bug fix?
--
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]
