Re: [OpenJDK 2D-Dev] [9] request for review: 8085910: OGL text renderer: gamma lut cleanup

2015-06-05 Thread Sergey Bylokhov
Looks fine. On 05.06.15 18:43, Phil Race wrote: +1 -phil. On 06/05/2015 07:36 AM, Andrew Brygin wrote: Hello Sergey, and Phil, could you please review a fix for CR 8085910? Bug: https://bugs.openjdk.java.net/browse/JDK-8085910 Webrev: http://cr.openjdk.java.net/~bae/8085910/9/webrev.00/

Re: [OpenJDK 2D-Dev] [9] request for review: 8085910: OGL text renderer: gamma lut cleanup

2015-06-05 Thread Phil Race
+1 -phil. On 06/05/2015 07:36 AM, Andrew Brygin wrote: Hello Sergey, and Phil, could you please review a fix for CR 8085910? Bug: https://bugs.openjdk.java.net/browse/JDK-8085910 Webrev: http://cr.openjdk.java.net/~bae/8085910/9/webrev.00/ This fix just removes the code related to gamma LU

[OpenJDK 2D-Dev] [9] request for review: 8085910: OGL text renderer: gamma lut cleanup

2015-06-05 Thread Andrew Brygin
Hello Sergey, and Phil, could you please review a fix for CR 8085910? Bug: https://bugs.openjdk.java.net/browse/JDK-8085910 Webrev: http://cr.openjdk.java.net/~bae/8085910/9/webrev.00/ This fix just removes the code related to gamma LUTs from ogl text renderer. This code is not needed anymor