RFR (L, final): 8218626: Add detailed message to NullPointerException describing what is null.
Lindenmaier, Goetz
goetz.lindenmaier at sap.com
Mon Oct 14 13:46:56 UTC 2019
Hi everybody,
JEP 358 is targeted to jdk14.
If there are no objections, I'll push this change tomorrow or
on Wednesday.
I'll do one final pass of jdk/submit before pushing.
The final webrev:
http://cr.openjdk.java.net/~goetz/wr19/8218628-exMsg-NPE/21/
Best regards,
Goetz.
> -----Original Message-----
> From: mark.reinhold at oracle.com <mark.reinhold at oracle.com>
> Sent: Donnerstag, 3. Oktober 2019 00:41
> To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>
> Cc: hotspot-runtime-dev at openjdk.java.net; core-libs-dev at openjdk.java.net
> Subject: Re: RFR (L, final): 8218626: Add detailed message to
> NullPointerException describing what is null.
>
> 2019/10/2 5:45:10 -0700, goetz.lindenmaier at sap.com:
> > thanks for looking at my change! Can I add you as reviewer?
>
> Sure.
>
> >> This is very nice work! I especially appreciate the thorough tests.
> >
> > Thanks! But adapting the many tests to changed messages is
> > quite cumbersome :) Thanks for supplying the patch right away!
>
> Nothing that a little Emacs-fu can’t handle.
>
> >> ...
> >>
> >> I also noticed that the generated messages use single quotes (‘'’) to
> >> quote the names of fields, etc., rather than double quotes (‘"’).
> >
> > I'm fine with this, but I think hotspot uses "'" for citing code
> > quite consistently. E.g., have a look at
> >
> http://hg.openjdk.java.net/jdk/jdk/file/b25362cec8ce/src/hotspot/share/inter
> preter/linkResolver.cpp
> > This can easily be changed. I could do a separate change
> > for hotspot and change all uses of ' in exceptions to ". What
> > do you think?
>
> I think that’d be a fine clean-up task, for later.
>
> - Mark
More information about the core-libs-dev
mailing list