[8u] RFR: 8241829: Cleanup the code for PrinterJob on windows

[Alex Kashchenko]

On 5/5/21, Hohensee, Paul <hohensee at amazon.com> wrote:
> Lgtm.

Thanks for the review! Marked the issue for approval.

>
> Thanks,
> Paul
>
> -----Original Message-----
> From: jdk8u-dev <jdk8u-dev-retn at openjdk.java.net> on behalf of Alex
> Kashchenko <akashche at redhat.com>
> Date: Wednesday, May 5, 2021 at 8:08 AM
> To: jdk8u-dev <jdk8u-dev at openjdk.java.net>
> Subject: [8u] RFR: 8241829: Cleanup the code for PrinterJob on windows
>
> Hi,
>
> Please review the backport of JDK-8241829 to 8u:
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8241829
>
> Original change: https://hg.openjdk.java.net/jdk/jdk/rev/e101dba52342
>
> 11u backport:
> https://hg.openjdk.java.net/jdk-updates/jdk11u-dev/rev/5dc8950e91d5
>
> 8u webrev: https://cr.openjdk.java.net/~akasko/jdk8u/8241829/webrev.00/
>
> Patch doesn't apply cleanly because 8u doesn't include changes from
> JDK-8027771 (not public, change [1], some details [2]) in
> PrintServiceLookupProvider.java. Testing: checked that added test
> passes, ran jck:api/javax_print .
>
> [1] https://hg.openjdk.java.net/jdk-updates/jdk11u-dev/rev/7b9c1e1532cf
> [2] https://mail.openjdk.java.net/pipermail/jdk8u-dev/2019-June/009564.html
>
> --
> -Alex
>
>
>


-- 
-Alex