Hi Yasumasa,
Some comments. http://cr.openjdk.java.net/~ysuenaga/JDK-8165736/webrev.00/test/jdk/com/sun/tools/attach/StartManagementAgent.java.udiff.html - if (!ex.getMessage().contains("Invalid com.sun.management.jmxremote.port number")) { + if (!ex.getMessage().contains("For input string: \"apa\"")) { What is the motivation for this change? It seems, the original comparison is better. http://cr.openjdk.java.net/~ysuenaga/JDK-8165736/webrev.00/test/hotspot/jtreg/serviceability/dcmd/jvmti/AttachFailed/AttachException.java.html http://cr.openjdk.java.net/~ysuenaga/JDK-8165736/webrev.00/test/hotspot/jtreg/serviceability/dcmd/jvmti/AttachFailed/AttachIncorrectLibrary.java.html http://cr.openjdk.java.net/~ysuenaga/JDK-8165736/webrev.00/test/hotspot/jtreg/serviceability/dcmd/jvmti/AttachFailed/AttachNoEntry.java.html http://cr.openjdk.java.net/~ysuenaga/JDK-8165736/webrev.00/test/hotspot/jtreg/serviceability/dcmd/jvmti/AttachFailed/AttachReturnError.java.html 37 public void run(CommandExecutor executor) { 38 try{A space is missed after 'try'. It is odd that all test java classes define exactly the same methods: sharedObjectName(), jmx() and cli(). Would it better to defin a common base class with these methods? Otherwise, it looks good. Thank you for taking care about it! What tests did you run to make sure there are no regressions? Thanks, Serguei On 11/1/17 05:59, Yasumasa Suenaga wrote: PING: Could you review and sponsor it? |
- RFR: 8165736: Error message should be shown whe... Yasumasa Suenaga
- PING: RFR: 8165736: Error message should b... Yasumasa Suenaga
- Re: PING: RFR: 8165736: Error message ... serguei.spit...@oracle.com
- Re: PING: RFR: 8165736: Error mess... Yasumasa Suenaga
- Re: PING: RFR: 8165736: Error ... serguei.spit...@oracle.com
- Re: PING: RFR: 8165736: E... Yasumasa Suenaga
- Re: PING: RFR: 816573... serguei.spit...@oracle.com
- Re: PING: RFR: 81... Yasumasa Suenaga
- Re: PING: RFR: 81... serguei.spit...@oracle.com
- Re: PING: RFR: 81... serguei.spit...@oracle.com
- Re: PING: RFR: 81... Yasumasa Suenaga
- Re: PING: RFR: 81... David Holmes
- Re: PING: RFR: 81... serguei.spit...@oracle.com