Review request: 8055230: Rename attach provider implementation class
David Holmes
david.holmes at oracle.com
Tue Aug 26 05:09:57 UTC 2014
I like this change! :)
Looks good to me.
Thanks Mandy!
David
-----
On 26/08/2014 2:29 PM, Mandy Chung wrote:
> Webrev:
> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8055230/
>
> This patch renames the class name of attach provider implementation class
> to be the same for all platforms. This simplifies the build logic and
> removes the need for generating the service config file at build time.
>
> The files renamed are
> unix/classes/sun/tools/attach/${OS}VirtualMachine.java
> -> ${OS}/classes/sun/tools/attach/VirtualMachineImpl.java
> unix/classes/sun/tools/attach/${OS}AttachProvider.java
> -> ${OS}/classes/sun/tools/attach/VirtualMachineImpl.java
>
> ${OS}/classes/sun/tools/attach/${OS}VirtualMachine.java
> -> ${OS}/classes/sun/tools/attach/VirtualMachineImpl.java
> ${OS}/classes/sun/tools/attach/${OS}AttachProvider.java
> -> ${OS}/classes/sun/tools/attach/VirtualMachineImpl.java
>
> and also corresponding native files.
>
> Mandy
>
>
More information about the serviceability-dev
mailing list