RFR 8223349: [lworld] Reflection support on static <init> factory methods for inline types

John Rose john.r.rose at oracle.com
Fri May 31 19:46:31 UTC 2019


On May 31, 2019, at 12:13 PM, Roger Riggs <Roger.Riggs at oracle.com> wrote:
> 
> Hi,
> 
> A minor update to make the test code checking constructors more generic and used for
> Point, Line, and NonFlattenedValue.  It only checked the signatures, but does not invoke each constructor.
> 
> webrev:
> http://cr.openjdk.java.net/~rriggs/webrev-init-8223349-4/index.html

Reviewed again, except:  The new test looks fragile with
respect to declaration order, and (IIRC) method order
is not preserved by reflection.  This could fail spuriously
if c.getDeclaredConstructors reports a surprising new order.
Quick fix:  Add a sorting phase to the stream that produces
declaredSig.  Make sure your golden list is also sorted.





More information about the valhalla-dev mailing list