> On 8 Aug 2016, at 12:14, Patrick Reinhart <patr...@reini.net> wrote:
> 
> 
>> Am 08.08.2016 um 18:55 schrieb Alan Bateman <alan.bate...@oracle.com>:
>> 
>> 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.

Reply via email to