Re: [OpenJDK Rasterizer] Marlin #4

2015-11-20 Thread Laurent Bourgès
Jim, I am ok with your changes to use the new Unsafe and all others also. PS: I added recently the enableLogs flag to make Marlin silent to mimic Phil's approach to disable stdout logs 6 months ago. I am going to sleep now. Good luck & good night, Laurent Le 20 nov. 2015 23:53, "Jim Graham" a

Re: [OpenJDK Rasterizer] Marlin #4

2015-11-20 Thread Phil Race
OK. So it is fine as you have it. -phil. On 11/20/2015 01:11 PM, Jim Graham wrote: On 11/20/15 12:53 PM, Phil Race wrote: On 11/20/2015 08:50 AM, Jim Graham wrote: Here is the webrev for the remaining pre-integration tasks below. This includes: - turning off Marlin logging static final b

Re: [OpenJDK Rasterizer] Marlin #4

2015-11-20 Thread Jim Graham
On 11/20/15 12:53 PM, Phil Race wrote: On 11/20/2015 08:50 AM, Jim Graham wrote: Here is the webrev for the remaining pre-integration tasks below. This includes: - turning off Marlin logging static final boolean enableLogs = false; 34 // enable Logger 35 static final boolean u

[OpenJDK Rasterizer] hg: graphics-rasterizer/jdk9/jdk: Flip the default to Marlin as the default RenderingEngine and turn off verbose info on startup.

2015-11-20 Thread james . graham
Changeset: 0bea2cd1c9d4 Author:flar Date: 2015-11-20 13:14 -0800 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/0bea2cd1c9d4 Flip the default to Marlin as the default RenderingEngine and turn off verbose info on startup. ! src/java.desktop/share/classes/sun/java

Re: [OpenJDK Rasterizer] Marlin #4

2015-11-20 Thread Laurent Bourgès
Jim, It seems ok. Laurent Le 20 nov. 2015 17:50, "Jim Graham" a écrit : > > Here is the webrev for the remaining pre-integration tasks below. This includes: > > - turning off Marlin logging > - switching to Marlin as the default renderer > - adding a flag to print out which renderer is used on

Re: [OpenJDK Rasterizer] Marlin #4

2015-11-20 Thread Jim Graham
Here is the webrev for the remaining pre-integration tasks below. This includes: - turning off Marlin logging - switching to Marlin as the default renderer - adding a flag to print out which renderer is used on startup webrev: http://cr.openjdk.java.net/~flar/Marlin/Defaults/webrev.00/ Please

Re: [OpenJDK Rasterizer] Marlin #4

2015-11-20 Thread Jim Graham
I will be pushing this code cleanup changeset momentarily followed by one more push to accomplish all of the changing of the various defaults... Second code cleanup task - switching to jdk.internal.misc.Unsafe... webrev: http://cr.openjdk.java.net/~flar/Marlin/NewUnsafeClass.0/

[OpenJDK Rasterizer] hg: graphics-rasterizer/jdk9/jdk: Blessed modifier order cleanup.

2015-11-20 Thread james . graham
Changeset: db283932ccf5 Author:flar Date: 2015-11-20 07:35 -0800 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/db283932ccf5 Blessed modifier order cleanup. ! src/java.desktop/share/classes/sun/java2d/marlin/ArrayCache.java ! src/java.desktop/share/classes/sun/ja

Re: [OpenJDK Rasterizer] Marlin #4

2015-11-20 Thread Phil Race
On 11/20/2015 08:50 AM, Jim Graham wrote: Here is the webrev for the remaining pre-integration tasks below. This includes: - turning off Marlin logging static final boolean enableLogs = false; 34 // enable Logger 35 static final boolean useLogger = enableLogs && MarlinProperties

[OpenJDK Rasterizer] hg: graphics-rasterizer/jdk9/jdk: Switch to new version of jdk.internal.misc.Unsafe class.

2015-11-20 Thread james . graham
Changeset: d3f9db4effbd Author:flar Date: 2015-11-20 08:18 -0800 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/d3f9db4effbd Switch to new version of jdk.internal.misc.Unsafe class. ! src/java.desktop/share/classes/sun/java2d/marlin/MarlinCache.java ! src/java.de

Re: [OpenJDK Rasterizer] Marlin #4

2015-11-20 Thread Jim Graham
I will be pushing this momentarily along with a few other pushes to accomplish the list of pre-integration changes I mentioned below... First cleanup push is using the blessed modifier order script. webrev: http://cr.openjdk.java.net/~flar/Marlin/BlessedModifiers/webrev.4.3/ There should be no

[OpenJDK Rasterizer] hg: graphics-rasterizer/jdk9/jdk: Marlin Renderer #4.3

2015-11-20 Thread james . graham
Changeset: 7b841dc1d6d2 Author:lbourges Date: 2015-11-20 07:07 -0800 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/7b841dc1d6d2 Marlin Renderer #4.3 ! src/java.desktop/share/classes/sun/java2d/marlin/ArrayCache.java ! src/java.desktop/share/classes/sun/java2d/ma

Re: [OpenJDK Rasterizer] Marlin #4

2015-11-20 Thread Jim Graham
Hi Laurent, I ran a bunch of tests on 4.2 and saw no issues and the performance looked good. There were still some things that Ductus was faster on, but I just did a brief run of a few tests I cobbled together so I don't know how representative they are. Marlin beat Ductus on a number of te