RFR(XXS) : 8180004: jdk.test.lib.DynamicVMOption should be moved to jdk.test.lib.management (original) (raw)
Vladimir Ivanov vladimir.x.ivanov at oracle.com
Thu May 11 18:30:32 UTC 2017
- Previous message: RFR(XXS) : 8180183: Confusing javadoc comment to the getOutput(ProcessBuilder processBuilder) method of jdk.test.lib.process.ProcessTools
- Next message: RFR(S): 8180003: Remove sys/ prefix from poll.h and signal.h includes
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Reviewed.
Best regards, Vladimir Ivanov
On 5/10/17 12:05 AM, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev//8180004/webrev.00/index.html
8 lines changed: 1 ins; 0 del; 7 mod; Hi all, could you please review this tiny patch which moves jdk.test.lib.DynamicVMOption class to jdk.test.lib.management package and updates the tests which use it? j.t.l.DynamicVMOption uses classes from jdk.management module, so having it in common testlibrary package might cause redundant module dependencies. webrev: http://cr.openjdk.java.net/~iignatyev//8180004/webrev.00/index.html jbs: https://bugs.openjdk.java.net/browse/JDK-8180004 testing: :hotspotall Thanks, -- Igor
- Previous message: RFR(XXS) : 8180183: Confusing javadoc comment to the getOutput(ProcessBuilder processBuilder) method of jdk.test.lib.process.ProcessTools
- Next message: RFR(S): 8180003: Remove sys/ prefix from poll.h and signal.h includes
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]