> On 8 Aug 2016, at 12:14, Patrick Reinhart <[email protected]> wrote:
>
>
>> Am 08.08.2016 um 18:55 schrieb Alan Bateman <[email protected]>:
>>
>> On 08/08/2016 17:29, Patrick Reinhart wrote:
>>
>>> :
>>> I tried to integrate your suggested changes here:
>>> http://cr.openjdk.java.net/~reinhapa/reviews/8161230/ClassLoader_StreamMethods.01
>>>
>> I should have been clearer. A lazy implementation of
>> resources/systemResources methods won't throw any exceptions, instead any
>> I/O exceptions will be wrapped with an UncheckedIOException and then thrown
>> from the method that caused the access to take place. There are several
>> examples of this already. For the javadoc then this will be described in the
>> method description rather than a @throws.
>>
>> -Alan
>
> I hope that this version is more likely that what you meant…
>
> http://cr.openjdk.java.net/~reinhapa/reviews/8161230/ClassLoader_StreamMethods.02
>
> <http://cr.openjdk.java.net/~reinhapa/reviews/8161230/ClassLoader_StreamMethods.02>
>
Perhaps consider:
The loading of resources will occur when the returned stream is evaluated. If
the loading of resources
results in an {@code IOException} then the I/O exception is wrapped in an
{@link UncheckedIOException}
that is then thrown.
Instead of <tt>…</tt> use {@code … }
Paul.
