RFR: 8057776 Misc cleanups of the attach code
Staffan Larsen
staffan.larsen at oracle.com
Mon Sep 8 13:04:22 UTC 2014
Hi Volker,
Unfortunately I had already pushed the change. I filed a new bug for the AIX work: JDK-8057778 and the changes are below. If you can review them, I will push it soon.
Thanks and sorry I forgot aix,
/Staffan
diff --git a/src/jdk.attach/aix/classes/sun/tools/attach/AttachProviderImpl.java b/src/jdk.attach/aix/classes/sun/tools/attach/AttachProviderImpl.java
--- a/src/jdk.attach/aix/classes/sun/tools/attach/AttachProviderImpl.java
+++ b/src/jdk.attach/aix/classes/sun/tools/attach/AttachProviderImpl.java
@@ -28,8 +28,6 @@
import com.sun.tools.attach.VirtualMachine;
import com.sun.tools.attach.VirtualMachineDescriptor;
import com.sun.tools.attach.AttachNotSupportedException;
-import com.sun.tools.attach.spi.AttachProvider;
-
import java.io.IOException;
// Based on linux/classes/sun/tools/attach/AttachProviderImpl.java.
@@ -40,9 +38,6 @@
*/
public class AttachProviderImpl extends HotSpotAttachProvider {
- // perf counter for the JVM version
- private static final String JVM_VERSION = "java.property.java.vm.version";
-
public AttachProviderImpl() {
}
diff --git a/src/jdk.attach/aix/classes/sun/tools/attach/VirtualMachineImpl.java b/src/jdk.attach/aix/classes/sun/tools/attach/VirtualMachineImpl.java
--- a/src/jdk.attach/aix/classes/sun/tools/attach/VirtualMachineImpl.java
+++ b/src/jdk.attach/aix/classes/sun/tools/attach/VirtualMachineImpl.java
@@ -25,14 +25,12 @@
*/
package sun.tools.attach;
-import com.sun.tools.attach.VirtualMachine;
import com.sun.tools.attach.AgentLoadException;
import com.sun.tools.attach.AttachNotSupportedException;
import com.sun.tools.attach.spi.AttachProvider;
import java.io.InputStream;
import java.io.IOException;
import java.io.File;
-import java.util.Properties;
// Based on linux/classes/sun/tools/attach/VirtualMachineImpl.java.
On 8 sep 2014, at 14:48, Volker Simonis <volker.simonis at gmail.com> wrote:
> hi Staffen,
>
> the change looks good, but could you please also change src/jdk.attach/aix/classes/sun/tools/attach/AttachProviderImpl.java
>
> Thanks,
> Volker
>
>
>
> On Mon, Sep 8, 2014 at 2:45 PM, Staffan Larsen <staffan.larsen at oracle.com> wrote:
> Thank you, all.
>
> On 8 sep 2014, at 14:39, Daniel Fuchs <daniel.fuchs at oracle.com> wrote:
>
> > Hi Staffan,
> >
> > Looks good!
> >
> > -- daniel
> >
> > On 9/8/14 2:23 PM, Staffan Larsen wrote:
> >> Please review this simple fix for some small issues in the attach code:
> >> - Removed unused imports
> >> - Removed unused fields
> >> - Update @since for startManagementAgent() and startLocalManagementAgent() since these were just backported to 1.8
> >> - Simplified a for-loop in a test
> >>
> >> bug: https://bugs.openjdk.java.net/browse/JDK-8057776
> >> webrev: http://cr.openjdk.java.net/~sla/8057776/webrev.00/
> >>
> >> Thanks,
> >> /Staffan
> >>
> >
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20140908/5361fcb1/attachment.html>
More information about the serviceability-dev
mailing list