Re: [OpenJDK Rasterizer] Fwd: sun.java2D.pisces big memory usage (waste ?)
On 3/25/13 10:01 AM, Laurent Bourgès wrote: > Dear Donald, Dalibor and 2D-dev members, > I sent an email to 2d-dev and rasterizer mailing lists 2d-dev is the right mailing list for this kind of questions, I think. You may want to get in touch with Denis Lila from Red Hat, who has in the past worked on performance improvements in this area. cheers, dalibor topic -- Oracle <http://www.oracle.com> Dalibor Topic | Principal Product Manager Phone: +494089091214 | Mobile: +491737185961 Oracle Java Platform Group ORACLE Deutschland B.V. & Co. KG | Kühnehöfe 5 | 22761 Hamburg ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 München Registergericht: Amtsgericht München, HRA 95603 Geschäftsführer: Jürgen Kunz Komplementärin: ORACLE Deutschland Verwaltung B.V. Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 Geschäftsführer: Alexander van der Ven, Astrid Kepper, Val Maher Green Oracle <http://www.oracle.com/commitment> Oracle is committed to developing practices and products that help protect the environment
Re: [OpenJDK Rasterizer] Marlin renderer contribution for review
I took a look at the webrev, and I have no objections, other than that src/java.desktop/share/classes/sun/java2d/marlin/FastMath.java seems to be based off third party code with an unknown license. So that class should be removed in the next iteration. It may be worth filing a bug to optimize Math.* performance in general with respect to known classes of inputs instead. Alternatively, StrictMath versions of these methods should be in pure Java, and avoid the JNI invocation penalty, so could be worth a try as well. cheers, dalibor topic On 19.03.2015 00:25, Laurent Bourgès wrote: Jim, Here is the new webrev: http://cr.openjdk.java.net/~lbourges/marlin/marlin.3/ Changes: - discussed changes in Dasher, Stroker, TransformingPathConsumer2D - indentation, line breaks (continuation lines) - revert many float suffix in math operations (useless ? does the compiler make conversions ?) Hope it looks better ... Good night, Laurent -- <http://www.oracle.com> Dalibor Topic | Principal Product Manager Phone: +494089091214 | Mobile: +491737185961 ORACLE Deutschland B.V. & Co. KG | Kühnehöfe 5 | 22761 Hamburg ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 München Registergericht: Amtsgericht München, HRA 95603 Geschäftsführer: Jürgen Kunz Komplementärin: ORACLE Deutschland Verwaltung B.V. Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 Geschäftsführer: Alexander van der Ven, Astrid Kepper, Val Maher <http://www.oracle.com/commitment> Oracle is committed to developing practices and products that help protect the environment
Re: [OpenJDK Rasterizer] Marlin renderer contribution for review
On 27.03.2015 22:04, Jim Graham wrote: One thing I'd like to hear from Joe or Phil or Dalibor is if it is OK for us to check in the current version to the graphics-rasterizer-dev java.net repo The best time and place for due diligence around contributions is before they hit a repository. As long as it includes code of unknown pedigree/license, a proposed changeset can't go into an OpenJDK Project repository. My understanding is that Laurent has been working on an improved version without the questionable code. I'll take a look at his latest webrev on Monday morning my time. cheers, dalibor topic -- <http://www.oracle.com> Dalibor Topic | Principal Product Manager Phone: +494089091214 | Mobile: +491737185961 ORACLE Deutschland B.V. & Co. KG | Kühnehöfe 5 | 22761 Hamburg ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 München Registergericht: Amtsgericht München, HRA 95603 Geschäftsführer: Jürgen Kunz Komplementärin: ORACLE Deutschland Verwaltung B.V. Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 Geschäftsführer: Alexander van der Ven, Astrid Kepper, Val Maher <http://www.oracle.com/commitment> Oracle is committed to developing practices and products that help protect the environment
Re: [OpenJDK Rasterizer] Marlin renderer contribution for review
On 29.03.2015 16:45, Laurent Bourgès wrote: Hi Jim, Here is the new webrev: http://cr.openjdk.java.net/~lbourges/marlin/marlin.6/ Thanks, Laurent, my concerns on FloatMath have been resolved in this version. cheers, dalibor topic -- <http://www.oracle.com> Dalibor Topic | Principal Product Manager Phone: +494089091214 | Mobile: +491737185961 ORACLE Deutschland B.V. & Co. KG | Kühnehöfe 5 | 22761 Hamburg ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 München Registergericht: Amtsgericht München, HRA 95603 Geschäftsführer: Jürgen Kunz Komplementärin: ORACLE Deutschland Verwaltung B.V. Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 Geschäftsführer: Alexander van der Ven, Astrid Kepper, Val Maher <http://www.oracle.com/commitment> Oracle is committed to developing practices and products that help protect the environment
