RFR: 8313674: (fc) java/nio/channels/FileChannel/BlockDeviceSize.java should test for more block devices [v5]

Brian Burkhalter bpb at openjdk.org
Mon May 13 21:15:06 UTC 2024


On Mon, 13 May 2024 20:41:13 GMT, Inigo Mediavilla Saiz <duke at openjdk.org> wrote:

>>> The update to the test looks okay to me, and I think useful to have the test changed to check > 0 is good. It might be clearer to have it fail if <= 0 of course.
>>> 
>>> I assume you'll bump the end date in the copyright header.
>> 
>> 
>> 
>>>
>> 
>> Thanks @AlanBateman !
>> 
>> I've updated the copyright in: https://github.com/openjdk/jdk/pull/19021/commits/78af373b3df429ea2683293517f2c75b0c12b00e
>> 
>> And updated the failure condition to `<=` in: https://github.com/openjdk/jdk/pull/19021/commits/868a40fddb6d6167e00cc0dbfc39470c9cbf4985
>
>> @txominpelu You need to remove the integration blocker by changing the title of this PR to match the summary of the issue
>> 
>> ```
>> 8313674: (fc) java/nio/channels/FileChannel/BlockDeviceSize.java should test for more block devices
>> ```
> 
> Thanks @bplb , I've fixed the title of the PR.

@txominpelu I think you will need to reissue the `/integrate` command.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/19021#issuecomment-2108807684


More information about the nio-dev mailing list