commit a8b331ef24502e81cf8daace58ce123a0205c498
author David Pursehouse <dpursehouse@collab.net> Wed Jun 05 15:08:39 2019 +0900
committer David Pursehouse <dpursehouse@collab.net> Wed Jun 05 15:08:39 2019 +0900
tree 72e27bad77e677f8b15cb97b0ebca231d1d62768
parent 41c28a48ae5bf58e2dcf4228185a7ab680f0b522

DateFormatter: Reduce visibility of constructor used for tests

DateFormatter(Optional<TimeZone> fixedTz, Format format) is only used in
tests. Reduce its visibility and annotate as @VisibleForTesting.

Change-Id: I2b62d91669a86f2510e89869c377413b3a7bd197

java/com/google/gitiles/DateFormatter.java

diff --git a/java/com/google/gitiles/DateFormatter.java b/java/com/google/gitiles/DateFormatter.java
index b14483f..69e8a86 100644
--- a/java/com/google/gitiles/DateFormatter.java
+++ b/java/com/google/gitiles/DateFormatter.java
@@ -14,6 +14,7 @@
 
 package com.google.gitiles;
 
+import com.google.common.annotations.VisibleForTesting;
 import java.io.IOException;
 import java.text.DateFormat;
 import java.text.SimpleDateFormat;
@@ -63,7 +64,8 @@
   private final Optional<TimeZone> fixedTz;
   private final Format format;
 
-  public DateFormatter(Optional<TimeZone> fixedTz, Format format) {
+  @VisibleForTesting
+  protected DateFormatter(Optional<TimeZone> fixedTz, Format format) {
     this.fixedTz = fixedTz;
     this.format = format;
   }