On 26/08/2014 05:29, 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
On 26 Aug 2014, at 08:26, Alan Bateman alan.bate...@oracle.com wrote:
On 26/08/2014 05:29, 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.
Nice!
/Erik
On 2014-08-26 06:29, 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
Hi Mandy,
I'm seeing some small differences in the various VirtualMachineImpl.java
files, but if I'm not mistaken, all the new AttachProviderImpl.java look
all the same. I wonder if the patch could be further simplified by
moving AttachProviderImpl.java to jdk.attach/share/classes (unless
There are some differences in the AttachProvideImpl files. Most notably the
windows version is very different. Linux, Bsd and Aix are the same. The Solaris
version has a minor difference in the “public String type()” implementation.
The Linux, Bsd, Aix and Solaris versions could probably be
On 8/26/14 12:59 PM, Staffan Larsen wrote:
There are some differences in the AttachProvideImpl files. Most notably the
windows version is very different. Linux, Bsd and Aix are the same. The Solaris
version has a minor difference in the “public String type()” implementation.
The Linux, Bsd,
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
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
Ahh. The simplicity!
Looks good!
Thanks,
/Staffan
On 26 aug 2014, at 06:29, Mandy Chung mandy.ch...@oracle.com 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