Add an assertion to try to help debug #4225

cushon · Error Prone Team · commit 2bd7859957eb · 2024-01-03T08:56:49.000-08:00
PiperOrigin-RevId: 595414757
diff --git a/core/src/main/java/com/google/errorprone/bugpatterns/nullness/NullnessUtils.java b/core/src/main/java/com/google/errorprone/bugpatterns/nullness/NullnessUtils.java
@@ -38,6 +38,7 @@
 import static com.sun.source.tree.Tree.Kind.PARAMETERIZED_TYPE;
 import static com.sun.tools.javac.parser.Tokens.TokenKind.DOT;
 import static java.lang.Boolean.TRUE;
+import static java.util.Objects.requireNonNull;
 
 import com.google.auto.value.AutoValue;
 import com.google.common.collect.ImmutableSet;
@@ -260,6 +261,7 @@ static boolean hasExtraParameterForEnclosingInstance(MethodSymbol symbol) {
       return false;
     }
     ClassSymbol constructedClass = enclosingClass(symbol);
+    requireNonNull(constructedClass, symbol::toString);
     return enclosingClass(constructedClass) != null && !constructedClass.isStatic();
   }
 

Original file line number	Diff line number	Diff line change
`@@ -38,6 +38,7 @@`
`38`	`38`	`import static com.sun.source.tree.Tree.Kind.PARAMETERIZED_TYPE;`
`39`	`39`	`import static com.sun.tools.javac.parser.Tokens.TokenKind.DOT;`
`40`	`40`	`import static java.lang.Boolean.TRUE;`
	`41`	`+import static java.util.Objects.requireNonNull;`
`41`	`42`
`42`	`43`	`import com.google.auto.value.AutoValue;`
`43`	`44`	`import com.google.common.collect.ImmutableSet;`
`@@ -260,6 +261,7 @@ static boolean hasExtraParameterForEnclosingInstance(MethodSymbol symbol) {`
`260`	`261`	`return false;`
`261`	`262`	`}`
`262`	`263`	`ClassSymbol constructedClass = enclosingClass(symbol);`
	`264`	`+ requireNonNull(constructedClass, symbol::toString);`
`263`	`265`	`return enclosingClass(constructedClass) != null && !constructedClass.isStatic();`
`264`	`266`	`}`
`265`	`267`