Minor clarifications after CL 423822183.

RELNOTES=n/a PiperOrigin-RevId: 424346757
google · Jan 26, 2022 · 9066755 · 9066755
1 parent 89ad267
commit 9066755
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/android/guava/src/com/google/common/collect/ImmutableMap.java b/android/guava/src/com/google/common/collect/ImmutableMap.java
@@ -833,13 +833,13 @@ public final V getOrDefault(@CheckForNull Object key, @CheckForNull V defaultVal
      * pass a literal "null" should probably just use `get`, but I would expect other callers to
      * pass an expression that *might* be null. This could happen with:
      *
-     * - a `getFooOrDefault(@Nullable Foo defaultValue)` method that returns
+     * - a `getFooOrDefault(@CheckForNull Foo defaultValue)` method that returns
      *   `map.getOrDefault(FOO_KEY, defaultValue)`
      *
      * - a call that consults a chain of maps, as in `mapA.getOrDefault(key, mapB.getOrDefault(key,
      *   ...))`
      *
-     * So it make sense for the parameter (and thus the return type) to be @Nullable.
+     * So it makes sense for the parameter (and thus the return type) to be @CheckForNull.
      *
      * Two other points:
      *

diff --git a/guava/src/com/google/common/collect/ImmutableMap.java b/guava/src/com/google/common/collect/ImmutableMap.java
@@ -976,13 +976,13 @@ public final V getOrDefault(@CheckForNull Object key, @CheckForNull V defaultVal
      * pass a literal "null" should probably just use `get`, but I would expect other callers to
      * pass an expression that *might* be null. This could happen with:
      *
-     * - a `getFooOrDefault(@Nullable Foo defaultValue)` method that returns
+     * - a `getFooOrDefault(@CheckForNull Foo defaultValue)` method that returns
      *   `map.getOrDefault(FOO_KEY, defaultValue)`
      *
      * - a call that consults a chain of maps, as in `mapA.getOrDefault(key, mapB.getOrDefault(key,
      *   ...))`
      *
-     * So it make sense for the parameter (and thus the return type) to be @Nullable.
+     * So it makes sense for the parameter (and thus the return type) to be @CheckForNull.
      *
      * Two other points:
      *