RFR 8223349: [lworld] Reflection support on static <init> factory methods for inline types
Roger.Riggs at oracle.com
Fri May 31 19:47:13 UTC 2019
Good point, I should be using Set instead of List.
On 05/31/2019 03:46 PM, John Rose wrote:
> On May 31, 2019, at 12:13 PM, Roger Riggs <Roger.Riggs at oracle.com> wrote:
>> 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.
> 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