Hi Roger,
On 2020-06-08 17:36, Roger Riggs wrote:
Hi Claes,
The new method name "argumentMistmatch" should be "argumentMismatch".
("Mis" vs "Mist").
nice catch! Will fix that and run a sanity test before push.
The rest looks fine.
Thanks!
/Claes
Roger
On 6/8/20 10:51 AM, Claes Redestad wrote:
Hi,
JDK-8246152 triggered a failure in a JCK test that expects a NPE
rather than a StringConcatException. We can keep the old behavior,
while retaining most of performance by removing the now superfluous
Objects.requireNonNull calls from within the parse loop.
Also fixed an issue where we were accidentally producing the wrong
message in case of mismatching number of constants or arguments.
Bug: https://bugs.openjdk.java.net/browse/JDK-8246632
Webrev: http://cr.openjdk.java.net/~redestad/8246632/open.01/
Testing: tier1
Thanks!
/Claes
