<AWT Dev> RFR: 8247370: Clean up unused printing code in awt_PrintJob.cpp

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

<AWT Dev> RFR: 8247370: Clean up unused printing code in awt_PrintJob.cpp

Phil Race
This removes a long time un-used code path.

-------------

Commit messages:
 - 8247370: Clean up unused printing code in awt_PrintJob.cpp

Changes: https://git.openjdk.java.net/jdk/pull/3083/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3083&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8247370
  Stats: 29 lines in 1 file changed: 0 ins; 19 del; 10 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3083.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3083/head:pull/3083

PR: https://git.openjdk.java.net/jdk/pull/3083
Reply | Threaded
Open this post in threaded view
|

Re: <AWT Dev> RFR: 8247370: Clean up unused printing code in awt_PrintJob.cpp

Sergey Bylokhov-2
On Thu, 18 Mar 2021 22:59:57 GMT, Phil Race <[hidden email]> wrote:

> This removes a long time un-used code path.

Marked as reviewed by serb (Reviewer).

-------------

PR: https://git.openjdk.java.net/jdk/pull/3083
Reply | Threaded
Open this post in threaded view
|

Re: <AWT Dev> RFR: 8247370: Clean up unused printing code in awt_PrintJob.cpp

Prasanta Sadhukhan-2
In reply to this post by Phil Race
On Thu, 18 Mar 2021 22:59:57 GMT, Phil Race <[hidden email]> wrote:

> This removes a long time un-used code path.

Maybe we should put noreg-cleanup in JBS.

-------------

Marked as reviewed by psadhukhan (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/3083
Reply | Threaded
Open this post in threaded view
|

Re: <AWT Dev> RFR: 8247370: Clean up unused printing code in awt_PrintJob.cpp

Phil Race
On Fri, 19 Mar 2021 04:45:46 GMT, Prasanta Sadhukhan <[hidden email]> wrote:

> Maybe we should put noreg-cleanup in JBS.

done

-------------

PR: https://git.openjdk.java.net/jdk/pull/3083
Reply | Threaded
Open this post in threaded view
|

<AWT Dev> Integrated: 8247370: Clean up unused printing code in awt_PrintJob.cpp

Phil Race
In reply to this post by Phil Race
On Thu, 18 Mar 2021 22:59:57 GMT, Phil Race <[hidden email]> wrote:

> This removes a long time un-used code path.

This pull request has now been integrated.

Changeset: d41f7512
Author:    Phil Race <[hidden email]>
URL:       https://git.openjdk.java.net/jdk/commit/d41f7512
Stats:     29 lines in 1 file changed: 0 ins; 19 del; 10 mod

8247370: Clean up unused printing code in awt_PrintJob.cpp

Reviewed-by: serb, psadhukhan

-------------

PR: https://git.openjdk.java.net/jdk/pull/3083