Looks fine to me too.
Clemsns: All the 9XXXXXX numbers are incident numbers. jcheck will reject
them as bug numbers.
-phil.
On 9/4/2013 5:06 AM, Andrew Brygin wrote:
Hello Clemens,
the change looks fine to me.
BTW, I have converted the report 9006475 to a jdk bug:
http://bugs.sun.com/view_bug.do?bug_id=8024261
It should be visible in the bugs database shortly.
Thanks,
Andrew
On 9/4/2013 3:36 PM, Clemens Eisserer wrote:
Hi,
Please review my patch for 9006475 at:
http://cr.openjdk.java.net/~ceisserer/9006475/webrev.00/
<http://cr.openjdk.java.net/%7Eceisserer/9006475/webrev.00/>
Before this patch we were calling the native method
XRenderRectanglesNative() even in the case that there was only single
rectangle to be rendered (which is the case for Graphics.fillRect).
As XRenderRectanglesNative() uses GetPrimitiveArrayCritical, this
caused high overhead for small fillRect() oerations.
With this patch we call renderRectangle instead, which passes the
reactangle's coordinates down using JNI function parameters. On my
private Laptop this improved throughput of 1x1 fillRect calls by a
factor of 3.
Thanks, Clemens
