RFR  java.nio.file.Files.readAllBytes() does not read any data when Files.size() is 0
ivan.gerasimov at oracle.com
Mon Jul 29 14:26:17 UTC 2013
On 29.07.2013 14:27, Chris Hegarty wrote:
> On 29/07/2013 00:43, Alan Bateman wrote:
>> On 28/07/2013 15:08, Ivan Gerasimov wrote:
>>> Hello Alan and everyone else
>>> Thank you for review!
>>> Here's another version of webrev:
> The change looks fine to me, but then again I was happy with previous
> version too ;-)
Thank you, Chris!
>> Thanks for the updated the webrev, the expansion of the capacity is much
>> nicer now. As regards whether to double or increase by BUFFER_SIZE then
>> it's probably a coin toss. One could come up with cases where someone is
>> using this method on a file that is extended by another entity. It would
>> depend on the extend rate as to which approach would be more efficient
>> (clearly such cases are going to be problematic as file locking should
>> be used to coordinate the access).
>> As regards the comment on "Reads" vs "Read", that was more just I've
>> wasn't sure if you had intended to include this or not.
>> So I'm happy to sponsor this for you (assuming we are done).
More information about the core-libs-dev