RFR: 8332586: Avoid cloning empty arrays in java.lang.reflect.{Method,Constructor}

jengebr duke at openjdk.org
Thu May 30 19:30:12 UTC 2024


On Tue, 21 May 2024 13:49:18 GMT, jengebr <duke at openjdk.org> wrote:

> Improve `java/lang/reflect/Method.java`  by eliminating needless cloning of Class[0] instances.  This cloning is intended to prevent callers from changing array contents, but smany Methods have zero exceptions or zero parameters, and returning the original `Class[0]` is sufficient.

Thanks for the suggestion!  I'll definitely add coverage of the other instances in this package, but I don't see an obvious place to put a utility method.  Any suggestions?

New method added, PR renamed.

OCA process is progressing, once finalized I'll rerun checks and publish the PR.

OCA completed, ready for review.

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

PR Comment: https://git.openjdk.org/jdk/pull/19327#issuecomment-2122723979
PR Comment: https://git.openjdk.org/jdk/pull/19327#issuecomment-2122984663
PR Comment: https://git.openjdk.org/jdk/pull/19327#issuecomment-2127017202
PR Comment: https://git.openjdk.org/jdk/pull/19327#issuecomment-2140731037


More information about the core-libs-dev mailing list