RFR: 8345987: java.management has two Util.newObjectName methods (remove one) [v2]

Leonid Mesnik lmesnik at openjdk.org
Fri Dec 13 20:01:36 UTC 2024


On Thu, 12 Dec 2024 09:54:03 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

>> Unnecessary duplication of small utility method.  There should be only one Util.newObjectName().
>
> Kevin Walls has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> 
>  - Merge remote-tracking branch 'upstream/master' into 8345987_newObjectName
>  - static imports
>  - 8345987: java.management has two Util.newObjectName methods (remove one)

> Unnecessary duplication of small utility method.

> This one uses ObjectName.getInstance(name), which is defined in ObjectName.java to do: return new ObjectName(name). So these two are equivalent.

Sorry, I mistaken with signature. 
Thanks for describing.

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

PR Comment: https://git.openjdk.org/jdk/pull/22681#issuecomment-2542199902


More information about the serviceability-dev mailing list