Hi Adam,
On 1/14/19 9:10 AM, Adam Farley8 wrote:
As for the CSR process, I'm unfamiliar with it. I've modified the
comment for the unreflectSetter method in the webrev. Do let me know if
more needs to be done.
The CSR process is straight-forward. CSR FAQ [1] may help you get started.
As for the fix, John has suggested the spec wording for this issue. It's
fine with the unreflectField method to have the explicit check and throw
field.makeAccessException(...). The exception message could match the
proposed spec wording like:
if(isSetter && field.isStatic() && field.isFinal())
throw field.makeAccessException("static final field has no
write access", this);
You have a very long line and the throw statement can start in the next
line.
In addition, please include a test case for this change.
Thanks
Mandy
[1] https://wiki.openjdk.java.net/display/csr/CSR+FAQs
