Hi Jim, I finally got the Marlin rasterizer integrated into the JDK9 client repo. >
Excellent ! Thanks a lot for your work in the last days to make it happen. > I fat-fingered the first integration and forgot to include "-u lbourges" > so Phil and Kevin and I sat on the phone and created a backout/reapply fix > and pushed them with the proper attributions, so if you see an initial > changeset notice go out with "Author: flar", wait for the updates to come > through with the "backout" and "integrate Marlin per JEP 265" changesets to > see the proper attribution. > Thank you to have made extra work (rollback + patch again) to submit the Marlin patch with my author ID. > You currently are assigned to the JEP so it is up to you to mark it > integrated. I could assign it to myself and mark it, but I figured you > would want to do the honors. > You were right: I enjoyed marking the JEP 265 as integrated (and added a comment to point to the resolved bug Id). > Congratulations on all the hard work! > Thanks but I would like to thank you too (Jim, Phil, Kevin, SQE team, Dalibor) who supported me in this effort (code reviews, tests, JEP, discussions). It represents definitely a lot of time spend on the code, running benchmarks, making reviews and discussing by email ... I am looking forward the first jdk9 EA build to test it and I hope Marlin will rock and will satisfy OpenJDK & Oracle JDK users. Finally some work remains: - Marlin code cleanup (comments, logging, system properties, automated tests) - J2DBench MT patch Future enhancements in Marlin: - array cache cleanup (using "cache reference") - NaN / overflow handling - use double instead of float in the rendering pipeline (dasher, stroker) for both performance but also improved accuracy - implement efficient polygon clipping (supporting affine transforms ?) R&D: - improve AA quality by estimating high-precision pixel coverages based on analytical area computations (trapezoid) PS: You should update the openjdk wiki pages later: http://openjdk.java.net/projects/graphics-rasterizer/ http://openjdk.java.net/groups/2d/ "As documented on the group page referenced above, this work has been implemented using the *Pisces* renderer, but still needs work in several areas : - *Performance is not as good as the library it replaces.* - Fixed point is used with little or no overflow protection - *Support for the STROKE_CONTROL hint*" Best regards, Laurent
