RFR(S): 8233014: Enable ShowCodeDetailsInExceptionMessages per default.

[David Holmes]

On 27/04/2020 6:20 pm, Reingruber, Richard wrote:
> Hi Goetz,
> 
> Just a minor detail: the license check fails for the file SuppressMessagesTest.java
> 
> Try
> 
>    sh make/scripts/lic_check.sh -gpl $(hg log -v --rev tip |sed -n '/files:[[:space:]]*/s///p')

Why did it fail?

> With
> 
> --- a/test/hotspot/jtreg/runtime/exceptionMsgs/NullPointerException/SuppressMessagesTest.java
> +++ b/test/hotspot/jtreg/runtime/exceptionMsgs/NullPointerException/SuppressMessagesTest.java
> @@ -1,6 +1,6 @@
>   /*
> + * Copyright (c) 2019, 2020, SAP SE. All rights reserved.
>    * Copyright (c) 2019, 2020, Oracle and/or its affiliates. All rights reserved.
> - * Copyright (c) 2019, 2020 SAP SE. All rights reserved.
>    * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
>    *
>    * This code is free software; you can redistribute it and/or modify it
> 
> it succeeds. Note that I added a comma after 2020 also.

That doesn't seem to be the SAP format. (Though it is not 100% consistent.)

Cheers,
David

> Cheers, Richard.
> 
> -----Original Message-----
> From: hotspot-dev <hotspot-dev-bounces at openjdk.java.net> On Behalf Of Lindenmaier, Goetz
> Sent: Freitag, 24. April 2020 16:36
> To: 'hotspot-dev at openjdk.java.net' <hotspot-dev at openjdk.java.net>
> Subject: RFR(S): 8233014: Enable ShowCodeDetailsInExceptionMessages per default.
> 
> Hi,
> 
> I would like to enable JEP 358 Helpful NullPointerExceptions by default:
> 
> http://cr.openjdk.java.net/~goetz/wr19/8233014-enable_NPE_per_default/02/
> 
> Please review.
> 
> The change passed our testing and jdk-submit.
> 
> Best regards,
>   Goetz.
>