RFR (L, final): 8218626: Add detailed message to NullPointerException describing what is null.
Lindenmaier, Goetz
goetz.lindenmaier at sap.com
Tue Sep 17 09:23:18 UTC 2019
Hi Thomas,
thanks for pointing this out. I improved the placement
of the ResourceMarks.
Unfortunately, base() returns an immutable string, but
for trim_well_known_class_names this does not work.
So I'd propose this:
http://cr.openjdk.java.net/~goetz/wr19/8218628-exMsg-NPE/17-incremental/
http://cr.openjdk.java.net/~goetz/wr19/8218628-exMsg-NPE/17/
Best regards,
Goetz.
> -----Original Message-----
> From: Thomas Stüfe <thomas.stuefe at gmail.com>
> Sent: Dienstag, 17. September 2019 09:06
> To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>
> Cc: Hotspot dev runtime <hotspot-runtime-dev at openjdk.java.net>; Java Core
> Libs <core-libs-dev at openjdk.java.net>
> Subject: Re: RFR (L, final): 8218626: Add detailed message to
> NullPointerException describing what is null.
>
> Additionally, since 8224193, stringStream does not use RA anymore, so you do
> not need ResourceMarks for the backing buffer. 8224193 has been backported
> to 11, btw.
>
> On Mon, Sep 16, 2019 at 2:53 PM Thomas Stüfe <thomas.stuefe at gmail.com
> <mailto:thomas.stuefe at gmail.com> > wrote:
>
>
> Hi Goetz,
>
> not a full review, just a small suggestion. In jvm.cpp you could just
> access ss->base() instead of ss->as_string() since the internal buffer is already
> NULL terminated and result_string does not outlive the stringStream object.
> Also it misses including ostream.hpp.
>
> Cheers, Thomas
>
>
> On Tue, Sep 10, 2019 at 4:46 PM Lindenmaier, Goetz
> <goetz.lindenmaier at sap.com <mailto:goetz.lindenmaier at sap.com> > wrote:
>
>
> Hi,
>
> the subject should mention 8218628. (Not 8218626).
> Sorry for this!
>
> Best regards,
> Goetz.
>
> From: Lindenmaier, Goetz
> Sent: Dienstag, 10. September 2019 11:48
> To: 'Hotspot dev runtime' <hotspot-runtime-
> dev at openjdk.java.net <mailto:hotspot-runtime-dev at openjdk.java.net> >; Java
> Core Libs <core-libs-dev at openjdk.java.net <mailto:core-libs-
> dev at openjdk.java.net> >
> Subject: RFR (L, final): 8218626: Add detailed message to
> NullPointerException describing what is null.
>
> Hi,
>
> this is the implementation of JEP 358: Helpful
> NullPointerExceptions.
>
> The JEP is in status "Candidate". Coleen (many, many thanks!)
> ran
> it through the Oracle-internal processes. Now I please need
> final reviews
> for this change so that I can propose it to target jdk 14.
>
> JEP: https://bugs.openjdk.java.net/browse/JDK-8220715
> Enhancement: https://bugs.openjdk.java.net/browse/JDK-
> 8218628
> webrev: http://cr.openjdk.java.net/~goetz/wr19/8218628-
> exMsg-NPE/16/
>
> The change ran through a lot of testing, all jtreg and jck tests to
> name
> only some. The webrev points to patch
> http://cr.openjdk.java.net/~goetz/wr19/8218628-exMsg-
> NPE/16/enable_NPE_message.patch
> that enables the change by default, which was useful for
> testing to
> assure the code is used in the tests.
> I just pushed the change to jdk/submit once more.
>
> Please review.
>
> Thanks!
> Goetz.
>
More information about the core-libs-dev
mailing list