RFR: 8336934: Clean up JavaLangReflectAccess
Roger Riggs
rriggs at openjdk.org
Thu Jul 25 21:59:31 UTC 2024
On Tue, 23 Jul 2024 04:10:38 GMT, Chen Liang <liach at openjdk.org> wrote:
> Removed redundant APIs in `JavaLangReflectAccess` and added general warning against using `SharedSecrets`.
src/java.base/share/classes/java/lang/reflect/Constructor.java line 168:
> 166: }
> 167:
> 168: /** Creates a new root constructor with a custom accessor for serialization hooks. */
Preserve "/**" comments for generated javadoc, a "//" comment would be more appropriate for a 1-liner.
src/java.base/share/classes/jdk/internal/access/JavaLangReflectAccess.java line 53:
> 51: //</editor-fold>
> 52:
> 53: //<editor-fold defaultstate="collapsed" desc="AccessibleObject Copying">
Personal editor comments should not be included.
src/java.base/share/classes/jdk/internal/access/JavaLangReflectAccess.java line 68:
> 66:
> 67: //</editor-fold>
> 68:
Remove
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20290#discussion_r1692174982
PR Review Comment: https://git.openjdk.org/jdk/pull/20290#discussion_r1692164266
PR Review Comment: https://git.openjdk.org/jdk/pull/20290#discussion_r1692165039
More information about the core-libs-dev
mailing list