RFR: JDK-8216558: Lookup.unreflectSetter(Field) fails to throw IllegalAccessException for final fields

Mandy Chung mandy.chung at oracle.com
Fri Mar 8 07:54:53 UTC 2019



On 3/7/19 3:17 PM, Mandy Chung wrote:
> 
>> Lastly, I'm sorry to see you weren't happy with the quality of my test
>> code. Your changes seem much larger in scale, and quite different to
>> my changes. Could you explain the benefits of your approach, vs simply
>> adding non-static final fields to my code?
> 
> First, I dont see my test update is large scale.  Second, it's not
> about the quality of the test code.  The size of the change is not
> the sole factor to consider but that seems to bother you.  Anyway,
> the test itself isn't easy to work with.  You have done the fix to
> include the regression test which is okay.

BTW one other main purpose I propose my patch is to show you an example
that proposing this change (that you consider larger in scale) does not
take months to approve.  I should make this explicit.

Mandy


More information about the core-libs-dev mailing list