errorprone :: ProtectedMembersInFinalClass (#794)

NationalSecurityAgency · May 29, 2024 · d613ccd · d613ccd
1 parent 4e9178d
commit d613ccd
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/main/java/emissary/core/channels/FileChannelFactory.java b/src/main/java/emissary/core/channels/FileChannelFactory.java
@@ -66,7 +66,7 @@ private LazyFileChannelImpl(final Path path) {
             this.path = path;
         }
 
-        protected void initialiseChannel() throws IOException {
+        void initialiseChannel() throws IOException {
             if (channel == null) {
                 channel = FileChannel.open(path, OPTIONS);
             }

diff --git a/src/main/java/emissary/pickup/WorkBundle.java b/src/main/java/emissary/pickup/WorkBundle.java
@@ -220,7 +220,7 @@ public String getBundleId() {
      * 
      * @return the new id value
      */
-    protected static String generateId() {
+    static String generateId() {
         return UUID.randomUUID().toString();
     }
 
@@ -325,7 +325,7 @@ public int addWorkUnit(WorkUnit workUnit, long fileModificationTimeInMillis, lon
      * @throws IllegalStateException if adding the units would cause the bundle to contain more than <code>MAX_UNITS</code>
      *         work units
      */
-    protected int addWorkUnits(List<WorkUnit> list) { // This appears to only be used by unit tests and the copy constructor
+    int addWorkUnits(List<WorkUnit> list) { // This appears to only be used by unit tests and the copy constructor
         if (workUnitList.size() + list.size() > MAX_UNITS) {
             throw new IllegalStateException("WorkBundle may not contain more than " + MAX_UNITS + " WorkUnits.");
         }
@@ -390,7 +390,7 @@ public int addFileName(String file, long fileModificationTimeInMillis, long file
      * @throws IllegalStateException if adding the files would cause the bundle to contain more than <code>MAX_UNITS</code>
      *         work units
      */
-    protected int addFileNames(String[] file) { // This appears to only be used by unit tests
+    int addFileNames(String[] file) { // This appears to only be used by unit tests
         for (String f : file) {
             addWorkUnit(new WorkUnit(f));
         }
@@ -404,8 +404,8 @@ protected int addFileNames(String[] file) { // This appears to only be used by u
      * @throws IllegalStateException if adding the files would cause the bundle to contain more than <code>MAX_UNITS</code>
      *         work units
      */
-    protected int addFileNames(List<String> list) { // This appears to only be used by unit tests and the copy
-                                                    // constructor
+    int addFileNames(List<String> list) { // This appears to only be used by unit tests and the copy
+                                          // constructor
         for (String file : list) {
             addWorkUnit(new WorkUnit(file));
         }
@@ -422,7 +422,7 @@ public int size() {
     /**
      * Clear the files from the list
      */
-    protected void clearFiles() {
+    void clearFiles() {
         // This is only used for testing
         workUnitList.clear();
         oldestFileModificationTime = Long.MAX_VALUE;