]> WPIA git - gigi.git/commitdiff
fix: Include visibility and read-only specifications wherever sensible.
authorBenny Baumann <BenBE@geshi.org>
Fri, 5 Jun 2015 17:01:29 +0000 (19:01 +0200)
committerFelix Dörre <felix@dogcraft.de>
Sun, 2 Aug 2015 18:48:05 +0000 (20:48 +0200)
12 files changed:
src/org/cacert/gigi/database/GigiPreparedStatement.java
src/org/cacert/gigi/database/GigiResultSet.java
src/org/cacert/gigi/database/SQLFileManager.java
src/org/cacert/gigi/dbObjects/CACertificate.java
src/org/cacert/gigi/dbObjects/CertificateOwner.java
src/org/cacert/gigi/dbObjects/CertificateProfile.java
src/org/cacert/gigi/dbObjects/Digest.java
src/org/cacert/gigi/dbObjects/Domain.java
src/org/cacert/gigi/dbObjects/ObjectCache.java
src/org/cacert/gigi/dbObjects/SupportedUser.java
src/org/cacert/gigi/dbObjects/User.java
src/org/cacert/gigi/dbObjects/Verifyable.java

index 55ed6ad3d53d003ae6639c575dc19147a36cdfd0..6d967648e9695a0c0e8781c95888b571904a4bed 100644 (file)
@@ -8,7 +8,7 @@ import java.sql.Timestamp;
 
 public class GigiPreparedStatement {
 
-    PreparedStatement target;
+    private PreparedStatement target;
 
     public GigiPreparedStatement(PreparedStatement preparedStatement) {
         target = preparedStatement;
@@ -106,4 +106,5 @@ public class GigiPreparedStatement {
         // TODO Auto-generated method stub
 
     }
+
 }
index 28fda28a4c8d783a24d97790fe13e680aa6882eb..d31c9ccb78ef83079c16eb02f8c04bc47d984536 100644 (file)
@@ -8,7 +8,7 @@ import java.sql.Timestamp;
 
 public class GigiResultSet implements Closeable {
 
-    ResultSet target;
+    private ResultSet target;
 
     public GigiResultSet(ResultSet target) {
         this.target = target;
index e51e52165d5c0c986623dd5f245c008601a3dd40..9aaebfae883c5ead9ff40d14a29846dac7f10095 100644 (file)
@@ -12,11 +12,11 @@ public class SQLFileManager {
 
     public static enum ImportType {
         /**
-         * Execute Script as-as
+         * Execute Script as-is
          */
         PRODUCTION,
         /**
-         * Execute Script, but changing Engine=InnoDB to Engine=Memory
+         * Execute Script, but change Engine=InnoDB to Engine=Memory
          */
         TEST,
         /**
index aa12d08f08b4d0ce400a6ebd8c3c4287162be97d..b6ede830e902887f8d58dcb34ba44fbebdb73e76 100644 (file)
@@ -19,15 +19,15 @@ import org.cacert.gigi.database.GigiResultSet;
 
 public class CACertificate implements IdCachable {
 
-    final String keyname;
+    private final String keyname;
 
-    final int id;
+    private final int id;
 
-    CACertificate parent = null;
+    private CACertificate parent = null;
 
-    final X509Certificate cert;
+    private final X509Certificate cert;
 
-    final String link;
+    private final String link;
 
     private CACertificate(int id) {
         this.id = id;
@@ -172,4 +172,5 @@ public class CACertificate implements IdCachable {
     public boolean isSelfsigned() {
         return this == getParent();
     }
+
 }
index 132b2787bd8c44b8d9c40a58a1024320570be891..5be24fc8562ef7615cd4e5863c37040f3a277fa8 100644 (file)
@@ -8,7 +8,7 @@ import org.cacert.gigi.database.GigiResultSet;
 
 public abstract class CertificateOwner implements IdCachable {
 
-    private static ObjectCache<CertificateOwner> myCache = new ObjectCache<>();
+    private static final ObjectCache<CertificateOwner> myCache = new ObjectCache<>();
 
     private int id;
 
index 43c1dd5ca3831d70161b25f23d6a7a03ff49d790..d9a9e34f170383fe177f8aebfbebba94717e6fbf 100644 (file)
@@ -22,9 +22,9 @@ public class CertificateProfile implements IdCachable {
 
     private final String visibleName;
 
-    private static HashMap<String, CertificateProfile> byName = new HashMap<>();
+    private static final HashMap<String, CertificateProfile> byName = new HashMap<>();
 
-    private static HashMap<Integer, CertificateProfile> byId = new HashMap<>();
+    private static final HashMap<Integer, CertificateProfile> byId = new HashMap<>();
 
     private final Map<String, PropertyTemplate> pt;
 
@@ -252,4 +252,5 @@ public class CertificateProfile implements IdCachable {
         }
         return true;
     }
+
 }
index edd8dcad907dd54d7fce1a5952aae09f541fcefd..01b7c5410a234a18161dc6327d60c7f4ff6b7234 100644 (file)
@@ -16,4 +16,5 @@ public enum Digest {
     public static Digest getDefault() {
         return SHA256;
     }
+
 }
index 00f51c1fe87df68cf050dd3020c49bbc7e245f9f..7e35225f4d99d409cee53e9aec64544c8563df9a 100644 (file)
@@ -69,6 +69,7 @@ public class Domain implements IdCachable, Verifyable {
     private int id;
 
     private static final Set<String> IDNEnabledTLDs;
+
     static {
         Properties CPS = new Properties();
         try (InputStream resourceAsStream = Domain.class.getResourceAsStream("CPS.properties")) {
@@ -261,7 +262,7 @@ public class Domain implements IdCachable, Verifyable {
 
     }
 
-    private static ObjectCache<Domain> myCache = new ObjectCache<>();
+    private static final ObjectCache<Domain> myCache = new ObjectCache<>();
 
     public static synchronized Domain getById(int id) throws IllegalArgumentException {
         Domain em = myCache.get(id);
index d1e41f82e3baaa0b359e6d55b1283fb81e0b85c4..c17dcacff4095ce666bb825a7d1e8394b5b5125b 100644 (file)
@@ -6,9 +6,9 @@ import java.util.HashSet;
 
 public class ObjectCache<T extends IdCachable> {
 
-    private HashMap<Integer, WeakReference<T>> hashmap = new HashMap<>();
+    private final HashMap<Integer, WeakReference<T>> hashmap = new HashMap<>();
 
-    private static HashSet<ObjectCache<?>> caches = new HashSet<>();
+    private static final HashSet<ObjectCache<? extends IdCachable>> caches = new HashSet<>();
 
     protected ObjectCache() {
         caches.add(this);
@@ -27,7 +27,7 @@ public class ObjectCache<T extends IdCachable> {
     }
 
     public static void clearAllCaches() {
-        for (ObjectCache<?> objectCache : caches) {
+        for (ObjectCache<? extends IdCachable> objectCache : caches) {
             objectCache.hashmap.clear();
         }
     }
index 447825ba8eeb3c81b161d3aaa71bd31152a02d88..17d7c8cf56984d39b985ac86554c9067a93f1be6 100644 (file)
@@ -9,7 +9,9 @@ import org.cacert.gigi.dbObjects.Certificate.CertificateStatus;
 
 public class SupportedUser {
 
-    private User target, supporter;
+    private User target;
+
+    private User supporter;
 
     private String ticket;
 
index 42457db543a625540be62b5deff109321fd72778..911d98b75c34d8184b522d85f2daddcd058b7966 100644 (file)
@@ -33,7 +33,7 @@ public class User extends CertificateOwner {
 
     private Locale locale;
 
-    private Set<Group> groups = new HashSet<>();
+    private final Set<Group> groups = new HashSet<>();
 
     protected User(GigiResultSet rs) {
         super(rs.getInt("id"));
index 68e7173ade924d0cef0372e86011209ae1c91dc1..e1c997e44ec4fbcb6298c9bef1225cab26ae4d2d 100644 (file)
@@ -5,4 +5,5 @@ import org.cacert.gigi.GigiApiException;
 public interface Verifyable {
 
     public void verify(String hash) throws GigiApiException;
+
 }