RFR: 8224974: Implement JEP 352
Dmitry Chuyko
dmitry.chuyko at bell-sw.com
Thu Aug 1 04:09:30 UTC 2019
Hi Andrew,
Just a small comment about the tests. As you can see, some of tests in
jdk/java/nio/channels/FileChannel check various modes, arguments, and
expected exceptions. E.g. see MapTest, Mode and Args.
I noticed that in your changes for the new map mode there are new
exceptions thrown in different situations. For example, when the
required mode is not supported, or it does not fit. In particular, this
should happen correctly on systems that do not support nvram. Have you
considered the possibility of extending or adding tests for this behavior?
-Dmitry
On 7/31/19 1:55 PM, Andrew Dinn wrote:
> Hi Aleksey
>
> On 30/07/2019 17:00, Aleksey Shipilev wrote:
>> On 7/30/19 5:04 PM, Andrew Dinn wrote:
>> ...........................
> Sure, welcome to the party even if most of the booze has been drunk :-)
>
> Thank you for a very thorough review. Comments (mostly in agreement) are
> inline below. They include a few refusals and a walk past some of the
> jumps pending confirmation they truly form part of the obstacle course.
>
> A new webrev against the latest jdk/jdk dev tree including all
> uncontested changes is here:
>
> http://cr.openjdk.java.net/~adinn/8224974/webrev.10/
>
>> === General:..................................
More information about the core-libs-dev
mailing list