Regression after jpackage+1-49 update

[Brent Christian]

On 10/7/19 1:12 AM, Alan Bateman wrote:
>> On 7/10/2019 4:43 pm, Alan Bateman wrote:
>>> On 07/10/2019 04:35, David Holmes wrote:
>>>> >>>> You can temporarily workaround with
>>>> -XX:+ClassForNameDeferLinking, but your code needs to be
>>>> updated to deal with LinkageErrors from Class.forName. >>>
>>> I suspect this code needs to translate NCDEF to
>>> TypeNotPresentException, other linkage errors maybe need to be
>>> handled here too. >
> I've created JDK-8231924 to track this.

Addressing JDK-8231924 will fix it so Field.getGenericType() throws a 
TypeNotPresentException, as specified.

I suspect that prior to the Class.forName() linking fix[1], there was no 
attempt to link the 'OptionalService' class (referring to Tornai's test 
case - it throws no exception with -XX:+ClassForNameDeferLinking).

It could be that the code would also need to be updated to link 
successfully, in order to avoid a TypeNotPresentException.

-Brent

1. https://bugs.openjdk.java.net/browse/JDK-8212117