On Fri, 5 Jun 2026 14:06:55 GMT, Chen Liang <[email protected]> wrote:
>> ReflectionFactory.newConstructorForSerializatoin might bypass strict >> initialization. We should make sure the skipped classes in the constructor >> chain do not declare strict instance fields. >> >> This updates the specification of jdk.unsupported's ReflectionFactory to >> reflect this behavioral update. >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Chen Liang has updated the pull request incrementally with one additional > commit since the last revision: > > Apply suggestions from code review > > Co-authored-by: Viktor Klang <[email protected]> Changes requested by [email protected] (no known OpenJDK username). src/jdk.unsupported/share/classes/sun/reflect/ReflectionFactory.java line 87: > 85: * Returns an accessible no-arg constructor for a class. > 86: * The no-arg constructor is found searching the class and its > supertypes, > 87: * and must ensure that all strictly-initialized fields Wrong indentation: Suggestion: * The no-arg constructor is found searching the class and its supertypes, * and must ensure that all strictly-initialized fields ------------- PR Review: https://git.openjdk.org/valhalla/pull/2350#pullrequestreview-4484394527 PR Review Comment: https://git.openjdk.org/valhalla/pull/2350#discussion_r3402310991
