Re: [OpenJDK 2D-Dev] [9] Review Request: 8164104 Cleanup of javaclient related mapfiles

2016-08-18 Thread Philip Race
On 8/17/16, 6:33 AM, Sergey Bylokhov wrote: Tested on solarisX64/solarisSparc/linuxX64 no issues were found(SwingSet2/Java2Demo). Sounds good. +1 But I wonder why the text on solarissparc filled by yellow shadow(Is it a known issue)? Yes there is an open bug. I believe it is a SPARC (or

Re: [OpenJDK 2D-Dev] [9] Review Request: 8164104 Cleanup of javaclient related mapfiles

2016-08-17 Thread Sergey Bylokhov
Tested on solarisX64/solarisSparc/linuxX64 no issues were found(SwingSet2/Java2Demo). But I wonder why the text on solarissparc filled by yellow shadow(Is it a known issue)? On 16.08.16 19:33, Phil Race wrote: I think "JPRT is OK" is good but may be not enough here? I am not sure if the build

Re: [OpenJDK 2D-Dev] [9] Review Request: 8164104 Cleanup of javaclient related mapfiles

2016-08-16 Thread Phil Race
I think "JPRT is OK" is good but may be not enough here? I am not sure if the build will complain about missing entries for JNI methods since there is no direct linking against these. Can you do a quick sanity run through of Java2Demo on Linux .. and ideally Solaris x64 too. -phil. On

[OpenJDK 2D-Dev] [9] Review Request: 8164104 Cleanup of javaclient related mapfiles

2016-08-16 Thread Sergey Bylokhov
Hello. Please review cleanup of mapfiles, initially the problem was found here: http://mail.openjdk.java.net/pipermail/awt-dev/2016-August/011734.html - tabs were replaced by spaces - doubled empty lines removed jprt is ok. Bug: https://bugs.openjdk.java.net/browse/JDK-8164104 Patch can be

[OpenJDK 2D-Dev] [9] Review Request: 8164104 Cleanup of javaclient related mapfiles

2016-08-16 Thread Sergey Bylokhov
Hello. Please review cleanup of mapfiles, initially the problem was found here: http://mail.openjdk.java.net/pipermail/awt-dev/2016-August/011734.html - tabs were replaced by spaces - doubled empty lines removed jprt is ok. Bug: https://bugs.openjdk.java.net/browse/JDK-8164104 Patch can be