Review request for JMC-6114: Fixing spotbugs violation (false positive) in LocalJVMToolkit

Sharath Ballal sharath.ballal at oracle.com
Fri Aug 17 15:06:30 UTC 2018


Hi Marcus,

Though vm may not be null all the time, if it is null we will still try to detach.  Is there a reason why you don’t want to check the vm for NULL before detaching ?

Thanks,
Sharath


-----Original Message-----
From: Jayathirth D V 
Sent: Thursday, August 16, 2018 3:39 PM
To: Marcus Hirt; jmc-dev at openjdk.java.net
Subject: RE: Review request for JMC-6114: Fixing spotbugs violation (false positive) in LocalJVMToolkit

Hi Marcus,

Changes are fine.
Before the change browser.attach test was causing "mvn verify" to fail.
After the change "mvn verify" passes.

Thanks,
Jay

-----Original Message-----
From: Marcus Hirt 
Sent: Monday, August 13, 2018 3:19 PM
To: jmc-dev at openjdk.java.net
Subject: Review request for JMC-6114: Fixing spotbugs violation (false positive) in LocalJVMToolkit

Hi all,

Please review this small fix to get the tests to run properly.

Jira:   https://bugs.openjdk.java.net/browse/JMC-6114
Webrev: http://cr.openjdk.java.net/~hirt/JMC-6114/webrev.01/

Kind regards,
Marcus




More information about the jmc-dev mailing list