On Fri, 5 Aug 2022 11:35:54 GMT, Tejesh R <t...@openjdk.org> wrote: >> `DebugGraphics` class has a Graphics instance which is been used in slowed >> down drawing. The `graphics` object is not initialized anywhere inside the >> class, where it is expected to set explicitly by the user. When the user >> doesn't set it and try to use the any mehtods like `drawing/setFont`, NPE is >> raised which is expected. The scenario is taken care by checking if the >> `graphics` object is null before using it inside the class, thus eliminating >> the NPE case. > > Tejesh R has updated the pull request incrementally with one additional > commit since the last revision: > > Updated based on review comments
src/java.desktop/share/classes/javax/swing/DebugGraphics.java line 89: > 87: > 88: // Creates a Graphics context when the constructor is called. > 89: if (graphics == null) { Probably it will be better to use `this.graphics` here and below while setting. ------------- PR: https://git.openjdk.org/jdk/pull/9673