RFR(s): 8073923: Files.lines() documentation needs clarification

Joseph D. Darcy joe.darcy at oracle.com
Tue Mar 3 02:46:15 UTC 2015


Hi Stuart,

The revised version looks good; thanks,

-Joe

On 3/2/2015 6:43 PM, Stuart Marks wrote:
> Hi Joe,
>
> Oh yes, good point. Revised webrev:
>
>     http://cr.openjdk.java.net/~smarks/reviews/8073923/webrev.1/
>
> Thanks,
>
> s'marks
>
>
> On 3/2/15 6:14 PM, Joseph D. Darcy wrote:
>> Hi Stuart,
>>
>> Since try-with-resources is "just" a desugaring, I think the 
>> statements like
>>
>>           [The resource] must be opened as a resource within a 
>> try-with-resources
>>   136  * statement to ensure that it is closed promptly after its 
>> operations
>> have completed.
>>
>> should be replaced with the milder
>>
>>      ... with a try-with-resources statement or similar control 
>> structure to
>> ensure...
>>
>> Thanks,
>>
>> -Joe
>>
>> On 3/2/2015 5:57 PM, Stuart Marks wrote:
>>> Hi all,
>>>
>>> Please review minor clarifications to Files.lines() and other Files 
>>> methods
>>> that return streams. The changes emphasize that try-with-resources 
>>> should be
>>> used. This also clarifies and strengthens a bit of the Stream class 
>>> doc that
>>> talks about when to use try-with-resources on streams.
>>>
>>> Bug:
>>>
>>>     https://bugs.openjdk.java.net/browse/JDK-8073923
>>>
>>> Webrev:
>>>
>>> http://cr.openjdk.java.net/~smarks/reviews/8073923/webrev.0/
>>>
>>> Thanks,
>>>
>>> s'marks
>>




More information about the core-libs-dev mailing list