RFR: 8256808: com/sun/jdi/CatchAllTest.java failed with "NullPointerException: Cannot invoke "lib.jdb.Jdb.log(String)" because "this.jdb" is null"

Serguei Spitsyn sspitsyn at openjdk.java.net
Mon Nov 30 23:53:55 UTC 2020


On Wed, 25 Nov 2020 21:36:12 GMT, Alex Menkov <amenkov at openjdk.org> wrote:

> JdbTest can get exception before jdb field is initialized.
> As Jdb logging does not depend on the instance, made Jdb.log method static

Hi Alex,
It looks good.
I do not object against of getting rid of the Jdb.log method if there are no other dependencies.
Thanks,
Serguei

-------------

Marked as reviewed by sspitsyn (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/1443


More information about the serviceability-dev mailing list