commit 947c7413524ab56bd426ab3cdc7b4d7755f3033b
author Dave Borowitz <dborowitz@google.com> Wed Oct 07 16:40:49 2015 -0400
committer Dave Borowitz <dborowitz@google.com> Thu Oct 08 10:17:37 2015 -0400
tree ce8f39ac7ee07f6f7a86a29654335633b841f707
parent 093f608b513e3818d04c3a82614d2e5b9e30b7b6

PatchSetInserter: Organize the myriad of fields

Change-Id: I91ee7b8ab876cd3729284fae4e47695c799b6d4d

gerrit-server/src/main/java/com/google/gerrit/server/change/PatchSetInserter.java

diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/change/PatchSetInserter.java b/gerrit-server/src/main/java/com/google/gerrit/server/change/PatchSetInserter.java
index cada143..4711d3a 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/change/PatchSetInserter.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/change/PatchSetInserter.java
@@ -87,6 +87,7 @@
     GERRIT, RECEIVE_COMMITS, NONE
   }
 
+  // Injected fields.
   private final ChangeHooks hooks;
   private final PatchSetInfoFactory patchSetInfoFactory;
   private final ReviewDb db;
@@ -97,15 +98,19 @@
   private final ApprovalCopier approvalCopier;
   private final ChangeMessagesUtil cmUtil;
 
+  // Assisted-injected fields.
   private final RevCommit commit;
   private final ChangeControl ctl;
   private final IdentifiedUser user;
   private final Repository git;
   private final RevWalk revWalk;
 
+  // Lazily initialized fields.
   private PatchSet.Id psId;
-  private String message;
+
+  // Fields exposed as setters.
   private SshInfo sshInfo;
+  private String message;
   private ValidatePolicy validatePolicy = ValidatePolicy.GERRIT;
   private boolean draft;
   private Iterable<String> groups;