cushon / issues-import

0 stars 0 forks source link

NPE in DeadException check #223

Closed cushon closed 9 years ago

cushon commented 9 years ago

Original issue created by alexeagle@google.com on 2013-12-20 at 01:23 AM


ran on class A {{ new RuntimeException(); }}

because there is no enclosing method, we get:

java.lang.NullPointerException at com.google.errorprone.matchers.Enclosing.findEnclosing(Enclosing.java:75) at com.google.errorprone.matchers.Enclosing.matches(Enclosing.java:68) at com.google.errorprone.matchers.Enclosing.matches(Enclosing.java:60) at com.google.errorprone.matchers.Matchers.matches(Matchers.java:100) at com.google.errorprone.matchers.Matchers.matches(Matchers.java:97) at com.google.errorprone.matchers.Matchers.matches(Matchers.java:112) at com.google.errorprone.matchers.Matchers.matches(Matchers.java:109) at com.google.errorprone.bugpatterns.DeadException.matchNewClass(DeadException.java:57) at com.google.errorprone.ErrorProneScanner.visitNewClass(ErrorProneScanner.java:710) at com.google.errorprone.ErrorProneScanner.visitNewClass(ErrorProneScanner.java:33) at com.sun.tools.javac.tree.JCTree.accept(JCTree.java:1388) at com.sun.source.util.TreePathScanner.scan(TreePathScanner.java:67)

cushon commented 9 years ago

Original comment posted by alexeagle@google.com on 2014-01-08 at 10:40 PM


Sigh, I can never get the issue tracker submit hooks to work.

Fixed by commit https://github.com/cushon/issues-import/commit/b2c3b7132868cb7b0f465ecc53808dcd9c0755b3


Status: Fixed