[OpenJDK 2D-Dev] Code review request to integrate IcedTea Color.createContext fix into OpenJDK 6
Joe Darcy
Joe.Darcy at Sun.COM
Fri Aug 8 03:10:56 UTC 2008
Hello.
I've made the requested edits, verified the JCK tests still pass, and
putback the change in OpenJDK 6 b12.
Thanks,
-Joe
Dmitri Trembovetski wrote:
>
>
> Hi Joe,
>
> A couple of comments:
>
> One is that the ColorModel check is not really needed
> since ColorPaintContext doesn't use the passed color model
> and in fact ignores it if you look at the code so
> there's no need to create a new one for different CM.
>
> So the color model check should be removed.
>
> Another is a minor stylistic one: Java2D code in general
> uses braces for all if statements, and 4 spaces offset.
>
> Also, for cases like this one with multiline conditions
> we put the opening curly brace on the new line (against
> the java style guideline) so that the next statement is
> easier to distinguish from the condition:
>
> if (multi
> line
> condition)
> {
> // stuff
> }
>
> Please change it to follow.
>
> Thanks,
> Dmitri
>
> Joe Darcy wrote:
>> Hello.
>>
>> Please review my application of the Color.createContext fix Keith
>> previously sent to the list
>>
>> http://mail.openjdk.java.net/pipermail/2d-dev/2008-April/000252.html
>>
>> to OpenJDK 6. I've done a build and the JCK tests in question now pass.
>>
>> Webrev uploaded to:
>> http://dcstaff.invokedynamic.info/images/1/1f/darcy-6735154.tgz
>>
>> Regards,
>>
>> -Joe
More information about the 2d-dev
mailing list