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.gra...@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

Reply via email to