[foreign] RFR: Fix UndefinedLayoutException exception message
Jorn Vernee
jbvernee at xs4all.nl
Mon Jan 28 13:16:41 UTC 2019
Yes, I will do that. (wasn't sure if it was useful enough).
Jorn
Sundararajan Athijegannathan schreef op 2019-01-28 14:12:
> Is it possible to add a test with invalid (unresolvable) name
> reference in a layout descriptor and check the exception message?
>
> -Sundar
>
> On 28/01/19, 6:36 PM, Jorn Vernee wrote:
>> Hi,
>>
>> JDK-8217245 changed the syntax for Unresolved layouts from `$(Name)`
>> to `${Name}`. This also means Unresolved now has a dedicated
>> `layoutExpression` field instead of relying on the name annotation.
>>
>> LayoutResolver.UndefinedLayoutException is still using the name
>> annotation in the exception message, which is now incorrect. This
>> small patch fixes that, and changes it to use
>> Unresolved::layoutExpression().
>>
>> Please review.
>>
>> Webrev:
>> http://cr.openjdk.java.net/~jvernee/panama/webrevs/fixmessage/webrev.00/
>>
>> Cheers,
>> Jorn
More information about the panama-dev
mailing list