On 4/6/16 3:52 PM, Alan Bateman wrote:
On 06/04/2016 08:47, Amy Lu wrote:
:
*
* @build Basic
+ * @modules java.base/jdk.internal.misc
* @run main CopyDirectMemory
*/
It might be clearer to put it before the @build, otherwise looks okay.
Yes :-)
Please review the updated patch:
http://cr.openjdk.java.net/~amlu/8153563/webrev.01/
Thanks,
Amy
--- old/test/java/nio/Buffer/Basic.java 2016-04-06 15:57:56.000000000 +0800
+++ new/test/java/nio/Buffer/Basic.java 2016-04-06 15:57:56.000000000 +0800
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2000, 2015, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2000, 2016, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -27,6 +27,7 @@
* 4526177 4463011 4660660 4661219 4663521 4782970 4804304 4938424 6231529
* 6221101 6234263 6535542 6591971 6593946 6795561 7190219 7199551 8065556
* 8149469
+ * @modules java.base/jdk.internal.misc
* @author Mark Reinhold
*/
--- old/test/java/nio/Buffer/CopyDirectMemory.java 2016-04-06 15:57:57.000000000 +0800
+++ new/test/java/nio/Buffer/CopyDirectMemory.java 2016-04-06
15:57:57.000000000 +0800
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2002, 2007, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2002, 2016, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -25,6 +25,7 @@
* @summary Test view buffer bulk operations for large buffers.
* @bug 4463011
*
+ * @modules java.base/jdk.internal.misc
* @build Basic
* @run main CopyDirectMemory
*/
