[OpenJDK 2D-Dev] Various fixes to pisces stroke widening code
Jim Graham
james.graham at oracle.com
Tue Aug 3 23:37:16 UTC 2010
Hi Denis,
I'd need to see a final webrev and approve it and then anyone with an
OpenJDK user id can push it. I'll also have to create a bugid for the
change set.
Are you a registered OpenJDK developer?
...jim
Denis Lila wrote:
> Hello Jim.
>
>> I'm guessing the test for "y == boundsMaxY-1" at line 470 could
>> probably also be deleted now (since it will be handled by the end
>> test when y reaches maxY)?
>
> Indeed it can. I've done this, and also updated a few comments that
> might have been misleading (they were written before certain changes
> were made).
>
>> It looks fine. Hopefully we can eventually figure out why the sorting
>> on the fly didn't pan out.
>
> Wonderful! So, will it be committed?
>
> Thanks,
> Denis.
>
> ----- "Jim Graham" <james.graham at oracle.com> wrote:
>
>> Hi Denis,
>>
>
>> Denis Lila wrote:
>>> Hi Jim.
>>>
>>> Thanks for all your suggestions. I fixed the edge array
>> indexing
>>> issue, the moveTo bug (not assigning x0), and the double
>>> initialization issue. I also improved the emission of the last row
>>> to what you said. The link is the same as the last one I sent.
>
>> But everything looks in order!
>>
>> ...jim
More information about the 2d-dev
mailing list