On Thu, 14 Oct 2021 14:44:34 GMT, Julia Boes <jb...@openjdk.org> wrote:
> sun.reflect.ReflectionFactory provides MethodHandles for the various > serialization methods, it is a critical internal API in the jdk.unsupported > module (see JEP 260 [1]) that may be used by 3rd party serialization > libraries. > > One of these serialization methods is readObjectNoData [2]: > > ```private void readObjectNoData() throws ObjectStreamException;``` > > The issue: The method that returns the matching handle, > sun.reflect.ReflectionFactory.readObjectNoDataForSerialization, uses an > erroneous signature `readObjectNoData(ObjectInputStream)` - note the > superfluous parameter. > > This change updates the specification and fixes the implementation in > java.base/jdk.internal.reflect.ReflectionFactory. > > Testing: tier 1-3 > > [1] https://openjdk.java.net/jeps/260 > [2] > https://docs.oracle.com/en/java/javase/15/docs/specs/serialization/input.html#the-readobjectnodata-method LGTM. The update on the `throws` declaration in the test is a bit of a distraction. ------------- Marked as reviewed by dfuchs (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/5951