RFR: 8260415: Remove unused class ReferenceProcessorMTProcMutator

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

RFR: 8260415: Remove unused class ReferenceProcessorMTProcMutator

Leo Korinth-2
ReferenceProcessorMTProcMutator is not used. ReferenceProcessorMTDiscoveryMutator seems to do the same and is still being used.

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

Commit messages:
 - 8260415

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

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

Re: RFR: 8260415: Remove unused class ReferenceProcessorMTProcMutator

Albert Mingkun Yang
On Mon, 15 Feb 2021 15:51:30 GMT, Leo Korinth <[hidden email]> wrote:

> ReferenceProcessorMTProcMutator is not used. ReferenceProcessorMTDiscoveryMutator seems to do the same and is still being used.

Marked as reviewed by ayang (Author).

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

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

Re: RFR: 8260415: Remove unused class ReferenceProcessorMTProcMutator

Stefan Johansson
In reply to this post by Leo Korinth-2
On Mon, 15 Feb 2021 15:51:30 GMT, Leo Korinth <[hidden email]> wrote:

> ReferenceProcessorMTProcMutator is not used. ReferenceProcessorMTDiscoveryMutator seems to do the same and is still being used.

Looks good and trivial.

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

Marked as reviewed by sjohanss (Reviewer).

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

Re: RFR: 8260415: Remove unused class ReferenceProcessorMTProcMutator

Leo Korinth-2
On Mon, 15 Feb 2021 20:39:16 GMT, Stefan Johansson <[hidden email]> wrote:

>> ReferenceProcessorMTProcMutator is not used. ReferenceProcessorMTDiscoveryMutator seems to do the same and is still being used.
>
> Looks good and trivial.

Thanks Albert and Stefan!

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

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

Integrated: 8260415: Remove unused class ReferenceProcessorMTProcMutator

Leo Korinth-2
In reply to this post by Leo Korinth-2
On Mon, 15 Feb 2021 15:51:30 GMT, Leo Korinth <[hidden email]> wrote:

> ReferenceProcessorMTProcMutator is not used. ReferenceProcessorMTDiscoveryMutator seems to do the same and is still being used.

This pull request has now been integrated.

Changeset: 61a659f4
Author:    Leo Korinth <[hidden email]>
URL:       https://git.openjdk.java.net/jdk/commit/61a659f4
Stats:     22 lines in 1 file changed: 0 ins; 22 del; 0 mod

8260415: Remove unused class ReferenceProcessorMTProcMutator

Reviewed-by: ayang, sjohanss

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

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