[ https://issues.apache.org/jira/browse/IO-868?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Gary D. Gregory updated IO-868: ------------------------------- Summary: Improve Javadoc for a BoundedInputStream builder() throwing IOException (was: Document better that a BoundedInputStream builder() can throw IOException) > Improve Javadoc for a BoundedInputStream builder() throwing IOException > ----------------------------------------------------------------------- > > Key: IO-868 > URL: https://issues.apache.org/jira/browse/IO-868 > Project: Commons IO > Issue Type: Wish > Components: Streams/Writers > Reporter: Julian Reschke > Assignee: Gary D. Gregory > Priority: Major > Fix For: 2.19.0 > > > See > > https://commons.apache.org/proper/commons-io/apidocs/org/apache/commons/io/input/BoundedInputStream.Builder.html#get() > This proved to be problematic when removing Guava usage from a project. > Guava's constructor for CountingInputStream doesn't declare a checked > exception, nor do the constructors of commons-io's CountingInputStream or > BoundedInputStream. > Using the recommended builder however does in the "get()" method. Why? > This makes it harder to convert existing code that does not expect an > exception here. What is the recommendation? Catch and wrap into a > RuntimeException? > Minimally it would good if the Javadoc explained under which condition the > IOException could occur. -- This message was sent by Atlassian Jira (v8.20.10#820010)