Marlin renderer patches for jdk8u integration

[Hohensee, Paul]

Hi, Laurent, I just approved the 8144630 backport.

The patches in the two directories are the same, btw.

Paul

From: Laurent Bourgès <bourges.laurent at gmail.com>
Date: Thursday, February 13, 2020 at 12:32 AM
To: "Hohensee, Paul" <hohensee at amazon.com>
Cc: Andrew Hughes <gnu.andrew at redhat.com>, jdk8u-dev <jdk8u-dev at openjdk.java.net>
Subject: Re: Marlin renderer patches for jdk8u integration

Dear Paul,
Thank you for being promptly back !

Le jeu. 13 févr. 2020 à 01:40, Hohensee, Paul <hohensee at amazon.com<mailto:hohensee at amazon.com>> a écrit :
Rather than go through a pile of separate reviews, here’s a review for all the ones for which Andrew asked a review.  I used the patches for the first 12 issues on the list at http://cr.openjdk.java.net/~lbourges/marlin-jdk8/marlin-jdk8.0/.
I suppose it is in fact:
http://cr.openjdk.java.net/~lbourges/marlin8u/marlin-8.03_to_12/

8143849: has been pushed to 8u.

8145055: approved by Andrew.

8144630: lgtm.

8144446: approved by Andrew.
8144445: approved by Andrew.

8144526: lgtm.

8144654: approved by Andrew.
8144718: approved by Andrew.

8149338: lgtm. The patch looks clean to me.

8148886: lgtm. The cleanerObj definition is in the diff context lines, not in the actual diffs, so the patch applies cleanly to RendererContext.java, net of context. 8147443 uses the Cleaner and associated classes that replaced finalizers. They haven’t been backported to 8u.

8144938: lgtm. I don’t see any differences in the diffs. :)

8159093: lgtm. I don’t see any context differences. RenderContext.java isn’t in the patch.
I will start one Official RFR thread per bug needing a review according to Andrew: 6 in total.

Please approve these RFR directly to follow the backport process.

Thank you in advance,
Laurent