Most of the pre-open-source changes are superseded by a single commit with the synopsis "Incorporate the latest pisces
improvements from OpenJDK" and the associated bug (which is marked confidential because this code was not open-sourced
at that time) says "Hey, Denis Lila did a whole lot of work on OpenPisces, we should integrate that into our Pisces!",
so I think most of the work prior to the bugs I point out below was already visible in the version of Pisces that you
started from in OpenJDK.
The only bug fixing that seemed to occur after those "update from JDK" changesets was to fix "leaks", but the associated
bug was more about transient garbage than true leaks and most of it was dealt with by reusing renderers and alpha
arrays, but you've already dug through that in your early Marlin->FX investigations.
So, I think the below list is reasonably comprehensive...
...jim
On 10/13/16 1:24 PM, Jim Graham wrote:
I'm trying to track down the old history after a lot of file moves, but here is
what I have so far:
(Note that some of these only apply when we use it in non-AA mode where we set
the number of subpixel samples to 1x1)
https://bugs.openjdk.java.net/browse/JDK-8092682 (non-AA: changing BND
constants)
https://bugs.openjdk.java.net/browse/JDK-8094493 (needed for non-AA, but
applicable to AA)
https://bugs.openjdk.java.net/browse/JDK-8094313 (first attempt to use for
both AA/non-AA)
https://bugs.openjdk.java.net/browse/JDK-8088013 (fixed, then backed out,
replaced by 8094313)
https://bugs.openjdk.java.net/browse/JDK-8097787 (likely the dash fixes you
noted below)
https://bugs.openjdk.java.net/browse/JDK-8119693 (AIOOB in dash array growth)
At that point, the files were moved from another repo during a huge
open-sourcing effort so I'll need a little more
digging to find further history.
Also, I don't think there is any history for "changes that were made when we ported
it from the JDK". I don't think
there were any "bug fixes" during that work for the initial commit, though, so
we can probably assume that the change
logs (when I can find the pre-open-source list) were the sum total of all of
the bug fixes...
...jim
On 10/13/16 1:05 PM, Phil Race wrote:
On 10/13/2016 12:22 PM, Laurent Bourgès wrote:
Hi,
I looked at openpisces enhancements in compare to pisces I forked in 2013.6 and
noticed at least 2 important changes:
- Dasher initialization: handle too small phase
- Stroker: new cases for parallel directions between control points in quad /
cubic curve subdivision
I plan to upgrade Marlin in jdk9 to integrate those improvements.
Is there OpenJFX bugs and tests to reproduce those complex situations ?
Between FX having multiple source code restructuring, open-sourcing losing
history and
a change from not always using bug ids in the commit through a change in
bug system .. and no strict requirements about regression tests it might be
challenging to identify the bugs under which such changes were made.
Maybe Jim can help a bit since he may have made or at least reviewed those
changes.
-phil.
Do you know other changes I should look at ?
PS: this work is more a backport of bug fixes than enhancements to be accepted
in jdk9, isn't it ?
I think it is still acceptable for 9.
-phil.
Thanks for your feedback,
Laurent
