Automatic code cleanup.

PiperOrigin-RevId: 648644801
google · Jul 2, 2024 · 2f6b8ff · 2f6b8ff
1 parent cf25065
commit 2f6b8ff
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 9 deletions.
diff --git a/...main/java/com/google/errorprone/bugpatterns/nullness/NullArgumentForNonNullParameter.java b/...main/java/com/google/errorprone/bugpatterns/nullness/NullArgumentForNonNullParameter.java
@@ -248,9 +248,7 @@ private boolean enclosingAnnotationDefaultsNonTypeVariablesToNonNull(
       if (hasAnnotation(sym, "com.google.protobuf.Internal$ProtoNonnullApi", state)) {
         return true;
       }
-      if ((hasAnnotation(sym, "org.jspecify.annotations.NullMarked", state)
-              // We break this string to avoid having it rewritten by Copybara.
-              || hasAnnotation(sym, "org.jspecify.null" + "ness.NullMarked", state))
+      if (hasAnnotation(sym, "org.jspecify.annotations.NullMarked", state)
           && weTrustNullMarkedOn(sym, state)) {
         return true;
       }

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
@@ -130,9 +130,7 @@ static boolean nullnessChecksShouldBeConservative(ErrorProneFlags flags) {
 
   static boolean isInNullMarkedScope(Symbol sym, VisitorState state) {
     for (; sym != null; sym = sym.getEnclosingElement()) {
-      if (hasAnnotation(sym, "org.jspecify.annotations.NullMarked", state)
-          // We break this string to avoid having it rewritten by Copybara.
-          || hasAnnotation(sym, "org.jspecify.null" + "ness.NullMarked", state)) {
+      if (hasAnnotation(sym, "org.jspecify.annotations.NullMarked", state)) {
         return true;
       }
     }
@@ -385,9 +383,6 @@ private static boolean isTypeUse(String className) {
       case "org.checkerframework.checker.nullness.qual.Nullable":
       case "org.jspecify.annotations.NonNull":
       case "org.jspecify.annotations.Nullable":
-      // We break these strings to avoid having them rewritten by Copybara.
-      case "org.jspecify.null" + "ness.NonNull":
-      case "org.jspecify.null" + "ness.Nullable":
         return true;
       default:
         // TODO(cpovirk): Detect type-use-ness from the class symbol if it's available?