commit 1889c3f06dedbdf8aab0d41e08156dcf52fef9ea
author Dave Borowitz <dborowitz@google.com> Tue Apr 04 11:31:57 2017 -0400
committer Dave Borowitz <dborowitz@google.com> Tue Apr 11 11:08:41 2017 -0400
tree cb80002435ebfb54e56c623241e42cc6ffdfd7b3
parent 83d1a98e078fca736b0d6a6ce80fc040830085b9

NoteDbChangeState: Mark some params @Nullable

Change-Id: If7dbf29c6cf4eb1f1d90867d5242d228747e1ff3

gerrit-server/src/main/java/com/google/gerrit/server/notedb/NoteDbChangeState.java

diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/notedb/NoteDbChangeState.java b/gerrit-server/src/main/java/com/google/gerrit/server/notedb/NoteDbChangeState.java
index fef7fdf..12967b8 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/notedb/NoteDbChangeState.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/notedb/NoteDbChangeState.java
@@ -78,11 +78,11 @@
       this.code = code;
     }
 
-    public static PrimaryStorage of(Change c) {
+    public static PrimaryStorage of(@Nullable Change c) {
       return of(NoteDbChangeState.parse(c));
     }
 
-    public static PrimaryStorage of(NoteDbChangeState s) {
+    public static PrimaryStorage of(@Nullable NoteDbChangeState s) {
       return s != null ? s.getPrimaryStorage() : REVIEW_DB;
     }
   }
@@ -150,12 +150,12 @@
     }
   }
 
-  public static NoteDbChangeState parse(Change c) {
+  public static NoteDbChangeState parse(@Nullable Change c) {
     return c != null ? parse(c.getId(), c.getNoteDbState()) : null;
   }
 
   @VisibleForTesting
-  public static NoteDbChangeState parse(Change.Id id, String str) {
+  public static NoteDbChangeState parse(Change.Id id, @Nullable String str) {
     if (Strings.isNullOrEmpty(str)) {
       // Return null rather than Optional as this is what goes in the field in
       // ReviewDb.