Hi,
On 11/22/18 12:40 AM, Laurent Bourgès wrote:
Hi sergey,
I proposed 2 small fixes for jdk12... still to be done, as better
solutions are not yet ready and I am completely overbusy on testing
Java Sorting algorithms.
Maybe I will give up fixing the native renderer, for these reasons:
- stdlib qsort implementation and performance (bentley qsort 93 or
merge sort on linux) is platform dependent, so it may take time to
validate its behaviour (small vs large arrays, not really random,
closer to almost sorted data)
I don't think we should use anyC++ stdlib:: code here if that is what
you mean.
Any small improvements we can make by having a better sort in our
sources would
be better.
If they aren't ready for 12, no problem. 13 is around the corner ..
- AA rendering is now the common case where the Marlin renderer is
active & fast enough.
Would you accept enabling antialiasing by default in jdk12 ?
Up to now, RenderingHint.ANTIALIASING_DEFAULT means ANTIALIASING_OFF.
That is a big change and I do not think we should do it.
-phil.
Several users faced performance issues with the native renderer
(hidpi, complex shapes, clipping dashed shapes) where the Marlin
renderer performs far better.
Finally I can now focus on a trivial Marlin patch to tune its sort
algorithm selection...
Cheers,
Laurent
Le jeu. 22 nov. 2018 à 00:04, Sergey Bylokhov
<sergey.bylok...@oracle.com <mailto:sergey.bylok...@oracle.com>> a écrit :
Hi, Laurent.
I do not think that we should postpone it to the next version.
Just send the changes when they are ready, if the review fails
to take place at the right time, then the fix will be
moved to the jdk13.
On 20/11/2018 00:28, Laurent Bourgès wrote:
> As OpenJDK12 RDP1 is coming soon, I propose this plan:
> - integrate this basic fix in ShapeSpanIterator.c code to use
stdlib sort (mergesort on linux)
> - integrate a very simple patch in Marlin renderer to disable
insertion sort for large arrays: 0.5s to 0.25s, few LOC
> - postpone my changes to Marlin sort & Marlin nonAA renderer
integration in OpenJDK 13
>
> Will you have time to review 2 small patchs on time ?