Re: jmx-dev RFR: 8257708: Remove redundant unmodifiableSet wrapper from already immutable set returned by Collections.singleton

2020-12-03 Thread Aleksey Shipilev
On Sun, 29 Nov 2020 18:03:24 GMT, Turbanov Andrey wrote: > 8257708: Remove redundant unmodifiableSet wrapper from already immutable set > returned by Collections.singleton Changes requested by shade (Reviewer). Submitted -- https://bugs.openjdk.java.net/browse/JDK-8257708, rename this PR to

jmx-dev RFR: 8257708: Remove redundant unmodifiableSet wrapper from already immutable set returned by Collections.singleton

2020-12-03 Thread Turbanov Andrey
8257708: Remove redundant unmodifiableSet wrapper from already immutable set returned by Collections.singleton - Commit messages: - 8257708: Remove redundant unmodifiableSet wrapper from already immutable set returned by Collections.singleton - [PATCH] Remove redundant unmodifiabl

Re: jmx-dev RFR: 8257708: Remove redundant unmodifiableSet wrapper from already immutable set returned by Collections.singleton

2020-12-03 Thread Serguei Spitsyn
On Sun, 29 Nov 2020 18:03:24 GMT, Andrey Turbanov wrote: > 8257708: Remove redundant unmodifiableSet wrapper from already immutable set > returned by Collections.singleton Hi Andrey, It looks good - nice simplification. Thanks, Serguei - Marked as reviewed by sspitsyn (Reviewer).

Re: jmx-dev RFR: 8257708: Remove redundant unmodifiableSet wrapper from already immutable set returned by Collections.singleton

2020-12-03 Thread Aleksey Shipilev
On Sun, 29 Nov 2020 18:03:24 GMT, Andrey Turbanov wrote: > 8257708: Remove redundant unmodifiableSet wrapper from already immutable set > returned by Collections.singleton Looks good now! - Marked as reviewed by shade (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/1505

jmx-dev Integrated: 8257708: Remove redundant unmodifiableSet wrapper from already immutable set returned by Collections.singleton

2020-12-03 Thread Andrey Turbanov
On Sun, 29 Nov 2020 18:03:24 GMT, Andrey Turbanov wrote: > 8257708: Remove redundant unmodifiableSet wrapper from already immutable set > returned by Collections.singleton This pull request has now been integrated. Changeset: d08c612b Author:Andrey Turbanov Committer: Aleksey Shipilev U