Hi Alan, thanks for the review .
> I think this looks okay except I think "emergency" should be dropped from the
> message (and the existing comment).
Are you referring to this “emergency” :
* First make our emergency fallback InternalError throwable.
*/
result = initializeFallbackError(jnienv);
- jplis_assert(result);
+ jplis_assert_msg(result, "emergency fallback init failed");
If you prefer I could output just “fallback init failed" - is this fine
with you?
Best regards, Matthias
From: Alan Bateman <alan.bate...@oracle.com>
Sent: Montag, 18. März 2019 10:09
To: Baesken, Matthias <matthias.baes...@sap.com>; Jean Christophe Beyler
<jcbey...@google.com>
Cc: serviceability-dev@openjdk.java.net
Subject: Re: RFR: 8220355: Improve assertion texts and exception messages in
eventHandlerVMInit
On 18/03/2019 08:54, Baesken, Matthias wrote:
Hi JC, thanks for your comments .
Second webrev , may I have a second review please ?
http://cr.openjdk.java.net/~mbaesken/webrevs/8220355.1/
I think this looks okay except I think "emergency" should be dropped from the
message (and the existing comment).
-Alan
