Fix inspections

diff --git a/src/main/java/org/asamk/signal/GroupNotFoundException.java b/src/main/java/org/asamk/signal/GroupNotFoundException.java
index 9f8c681af033de1c45195a5054f7f094037b35f9..5fee6233cb97289d1304d640ae5bd8dcbe8d27a8 100644 (file)
--- a/src/main/java/org/asamk/signal/GroupNotFoundException.java
+++ b/src/main/java/org/asamk/signal/GroupNotFoundException.java
@@ -5,10 +5,6 @@ import org.whispersystems.signalservice.internal.util.Base64;
 
 public class GroupNotFoundException extends DBusExecutionException {
 
 
 public class GroupNotFoundException extends DBusExecutionException {
 

-    public GroupNotFoundException(String message) {
-        super(message);
-    }
-

     public GroupNotFoundException(byte[] groupId) {
         super("Group not found: " + Base64.encodeBytes(groupId));
     }
     public GroupNotFoundException(byte[] groupId) {
         super("Group not found: " + Base64.encodeBytes(groupId));
     }

diff --git a/src/main/java/org/asamk/signal/NotAGroupMemberException.java b/src/main/java/org/asamk/signal/NotAGroupMemberException.java
index 42e021ec63f230bdf47b77412850629c16e02c34..84e2f90f8a912bb2f4727db08de3862c75ba7ce3 100644 (file)
--- a/src/main/java/org/asamk/signal/NotAGroupMemberException.java
+++ b/src/main/java/org/asamk/signal/NotAGroupMemberException.java
@@ -5,10 +5,6 @@ import org.whispersystems.signalservice.internal.util.Base64;
 
 public class NotAGroupMemberException extends DBusExecutionException {
 
 
 public class NotAGroupMemberException extends DBusExecutionException {
 

-    public NotAGroupMemberException(String message) {
-        super(message);
-    }
-

     public NotAGroupMemberException(byte[] groupId, String groupName) {
         super("User is not a member in group: " + groupName + " (" + Base64.encodeBytes(groupId) + ")");
     }
     public NotAGroupMemberException(byte[] groupId, String groupName) {
         super("User is not a member in group: " + groupName + " (" + Base64.encodeBytes(groupId) + ")");
     }

diff --git a/src/main/java/org/asamk/signal/UserAlreadyExists.java b/src/main/java/org/asamk/signal/UserAlreadyExists.java
index 047b5fc7ff7f12d98858265bb2759ab83a41ed46..28836f28bdf2eedce7dcc34624f55143dbb9cb55 100644 (file)
--- a/src/main/java/org/asamk/signal/UserAlreadyExists.java
+++ b/src/main/java/org/asamk/signal/UserAlreadyExists.java
@@ -2,8 +2,8 @@ package org.asamk.signal;
 
 public class UserAlreadyExists extends Exception {
 
 
 public class UserAlreadyExists extends Exception {
 

-    private String username;
-    private String fileName;
+    private final String username;
+    private final String fileName;

 
     public UserAlreadyExists(String username, String fileName) {
         this.username = username;
 
     public UserAlreadyExists(String username, String fileName) {
         this.username = username;

diff --git a/src/main/java/org/asamk/signal/manager/Manager.java b/src/main/java/org/asamk/signal/manager/Manager.java
index 1b703e4291a838b4454d329c4917458eba45d838..f8870a72d0cbbfacda3681da04435d3f76a51388 100644 (file)
--- a/src/main/java/org/asamk/signal/manager/Manager.java
+++ b/src/main/java/org/asamk/signal/manager/Manager.java
@@ -73,16 +73,14 @@ public class Manager implements Signal {
     private final String dataPath;
     private final String attachmentsPath;
     private final String avatarsPath;
     private final String dataPath;
     private final String attachmentsPath;
     private final String avatarsPath;

+    private final SleepTimer timer = new UptimeSleepTimer();

 
     private SignalAccount account;
 
     private SignalAccount account;

-

     private String username;
     private SignalServiceAccountManager accountManager;
     private SignalServiceMessagePipe messagePipe = null;
     private SignalServiceMessagePipe unidentifiedMessagePipe = null;
 
     private String username;
     private SignalServiceAccountManager accountManager;
     private SignalServiceMessagePipe messagePipe = null;
     private SignalServiceMessagePipe unidentifiedMessagePipe = null;
 

-    private SleepTimer timer = new UptimeSleepTimer();
-

     public Manager(String username, String settingsPath) {
         this.username = username;
         this.settingsPath = settingsPath;
     public Manager(String username, String settingsPath) {
         this.username = username;
         this.settingsPath = settingsPath;

diff --git a/src/main/java/org/asamk/signal/manager/Utils.java b/src/main/java/org/asamk/signal/manager/Utils.java
index f47dc1cefad67a8cb8985dcb9f206581d0e6df35..dbcd7e428f888df891c656c9fe3e2e14b5452dcb 100644 (file)
--- a/src/main/java/org/asamk/signal/manager/Utils.java
+++ b/src/main/java/org/asamk/signal/manager/Utils.java
@@ -223,8 +223,8 @@ class Utils {
 
     static class DeviceLinkInfo {
 
 
     static class DeviceLinkInfo {
 

-        String deviceIdentifier;
-        ECPublicKey deviceKey;
+        final String deviceIdentifier;
+        final ECPublicKey deviceKey;

 
         DeviceLinkInfo(final String deviceIdentifier, final ECPublicKey deviceKey) {
             this.deviceIdentifier = deviceIdentifier;
 
         DeviceLinkInfo(final String deviceIdentifier, final ECPublicKey deviceKey) {
             this.deviceIdentifier = deviceIdentifier;

diff --git a/src/main/java/org/asamk/signal/storage/contacts/JsonContactsStore.java b/src/main/java/org/asamk/signal/storage/contacts/JsonContactsStore.java
index 45e35e99526d11b78fc700ddc1c1de3b031846be..8d22550ba95947cd02516a6e80b6ebb426870b13 100644 (file)
--- a/src/main/java/org/asamk/signal/storage/contacts/JsonContactsStore.java
+++ b/src/main/java/org/asamk/signal/storage/contacts/JsonContactsStore.java
@@ -40,7 +40,7 @@ public class JsonContactsStore {
         contacts.clear();
     }
 
         contacts.clear();
     }
 

-    public static class MapToListSerializer extends JsonSerializer<Map<?, ?>> {
+    private static class MapToListSerializer extends JsonSerializer<Map<?, ?>> {

 
         @Override
         public void serialize(final Map<?, ?> value, final JsonGenerator jgen, final SerializerProvider provider) throws IOException {
 
         @Override
         public void serialize(final Map<?, ?> value, final JsonGenerator jgen, final SerializerProvider provider) throws IOException {


@@ -48,7 +48,7 @@ public class JsonContactsStore {
         }
     }
 
         }
     }
 

-    public static class ContactsDeserializer extends JsonDeserializer<Map<String, ContactInfo>> {
+    private static class ContactsDeserializer extends JsonDeserializer<Map<String, ContactInfo>> {

 
         @Override
         public Map<String, ContactInfo> deserialize(JsonParser jsonParser, DeserializationContext deserializationContext) throws IOException {
 
         @Override
         public Map<String, ContactInfo> deserialize(JsonParser jsonParser, DeserializationContext deserializationContext) throws IOException {

diff --git a/src/main/java/org/asamk/signal/storage/groups/JsonGroupStore.java b/src/main/java/org/asamk/signal/storage/groups/JsonGroupStore.java
index 6a3cdedbae7220ea2dc00655c143511813faa0b7..4c5677a693be86913e9e3ab77032f146d2b46fa4 100644 (file)
--- a/src/main/java/org/asamk/signal/storage/groups/JsonGroupStore.java
+++ b/src/main/java/org/asamk/signal/storage/groups/JsonGroupStore.java
@@ -37,7 +37,7 @@ public class JsonGroupStore {
         return new ArrayList<>(groups.values());
     }
 
         return new ArrayList<>(groups.values());
     }
 

-    public static class MapToListSerializer extends JsonSerializer<Map<?, ?>> {
+    private static class MapToListSerializer extends JsonSerializer<Map<?, ?>> {

 
         @Override
         public void serialize(final Map<?, ?> value, final JsonGenerator jgen, final SerializerProvider provider) throws IOException {
 
         @Override
         public void serialize(final Map<?, ?> value, final JsonGenerator jgen, final SerializerProvider provider) throws IOException {


@@ -45,7 +45,7 @@ public class JsonGroupStore {
         }
     }
 
         }
     }
 

-    public static class GroupsDeserializer extends JsonDeserializer<Map<String, GroupInfo>> {
+    private static class GroupsDeserializer extends JsonDeserializer<Map<String, GroupInfo>> {

 
         @Override
         public Map<String, GroupInfo> deserialize(JsonParser jsonParser, DeserializationContext deserializationContext) throws IOException {
 
         @Override
         public Map<String, GroupInfo> deserialize(JsonParser jsonParser, DeserializationContext deserializationContext) throws IOException {

diff --git a/src/main/java/org/asamk/signal/storage/protocol/JsonIdentityKeyStore.java b/src/main/java/org/asamk/signal/storage/protocol/JsonIdentityKeyStore.java
index a343ad4e4c9894a5638bf4f9d06d7c3f5a02d0c2..d7049e4a690d4bca56a2c077718987f36bee8e6b 100644 (file)
--- a/src/main/java/org/asamk/signal/storage/protocol/JsonIdentityKeyStore.java
+++ b/src/main/java/org/asamk/signal/storage/protocol/JsonIdentityKeyStore.java
@@ -2,7 +2,6 @@ package org.asamk.signal.storage.protocol;
 
 import com.fasterxml.jackson.core.JsonGenerator;
 import com.fasterxml.jackson.core.JsonParser;
 
 import com.fasterxml.jackson.core.JsonGenerator;
 import com.fasterxml.jackson.core.JsonParser;

-import com.fasterxml.jackson.core.JsonProcessingException;

 import com.fasterxml.jackson.databind.*;
 import org.asamk.signal.TrustLevel;
 import org.whispersystems.libsignal.IdentityKey;
 import com.fasterxml.jackson.databind.*;
 import org.asamk.signal.TrustLevel;
 import org.whispersystems.libsignal.IdentityKey;


@@ -189,13 +188,13 @@ public class JsonIdentityKeyStore implements IdentityKeyStore {
             this.added = new Date();
         }
 
             this.added = new Date();
         }
 

-        public Identity(IdentityKey identityKey, TrustLevel trustLevel, Date added) {
+        Identity(IdentityKey identityKey, TrustLevel trustLevel, Date added) {

             this.identityKey = identityKey;
             this.trustLevel = trustLevel;
             this.added = added;
         }
 
             this.identityKey = identityKey;
             this.trustLevel = trustLevel;
             this.added = added;
         }
 

-        public boolean isTrusted() {
+        boolean isTrusted() {

             return trustLevel == TrustLevel.TRUSTED_UNVERIFIED ||
                     trustLevel == TrustLevel.TRUSTED_VERIFIED;
         }
             return trustLevel == TrustLevel.TRUSTED_UNVERIFIED ||
                     trustLevel == TrustLevel.TRUSTED_VERIFIED;
         }

diff --git a/src/main/java/org/asamk/signal/storage/protocol/JsonPreKeyStore.java b/src/main/java/org/asamk/signal/storage/protocol/JsonPreKeyStore.java
index 3d4e21a36bfd3bf00a031db9cb2890f3ba49896b..3065bfde8127bf2ad852611a746f80ad5de52cbc 100644 (file)
--- a/src/main/java/org/asamk/signal/storage/protocol/JsonPreKeyStore.java
+++ b/src/main/java/org/asamk/signal/storage/protocol/JsonPreKeyStore.java
@@ -2,7 +2,6 @@ package org.asamk.signal.storage.protocol;
 
 import com.fasterxml.jackson.core.JsonGenerator;
 import com.fasterxml.jackson.core.JsonParser;
 
 import com.fasterxml.jackson.core.JsonGenerator;
 import com.fasterxml.jackson.core.JsonParser;

-import com.fasterxml.jackson.core.JsonProcessingException;

 import com.fasterxml.jackson.databind.*;
 import org.whispersystems.libsignal.InvalidKeyIdException;
 import org.whispersystems.libsignal.state.PreKeyRecord;
 import com.fasterxml.jackson.databind.*;
 import org.whispersystems.libsignal.InvalidKeyIdException;
 import org.whispersystems.libsignal.state.PreKeyRecord;


@@ -21,7 +20,7 @@ class JsonPreKeyStore implements PreKeyStore {
 
     }
 
 
     }
 

-    public void addPreKeys(Map<Integer, byte[]> preKeys) {
+    private void addPreKeys(Map<Integer, byte[]> preKeys) {

         store.putAll(preKeys);
     }
 
         store.putAll(preKeys);
     }
 

diff --git a/src/main/java/org/asamk/signal/storage/protocol/JsonSessionStore.java b/src/main/java/org/asamk/signal/storage/protocol/JsonSessionStore.java
index 87007d356657624de3803fb586437bcab8287983..7f5c6b0682d6fbc5bc7d5012183b7cc740a2b4d8 100644 (file)
--- a/src/main/java/org/asamk/signal/storage/protocol/JsonSessionStore.java
+++ b/src/main/java/org/asamk/signal/storage/protocol/JsonSessionStore.java
@@ -2,7 +2,6 @@ package org.asamk.signal.storage.protocol;
 
 import com.fasterxml.jackson.core.JsonGenerator;
 import com.fasterxml.jackson.core.JsonParser;
 
 import com.fasterxml.jackson.core.JsonGenerator;
 import com.fasterxml.jackson.core.JsonParser;

-import com.fasterxml.jackson.core.JsonProcessingException;

 import com.fasterxml.jackson.databind.*;
 import org.whispersystems.libsignal.SignalProtocolAddress;
 import org.whispersystems.libsignal.state.SessionRecord;
 import com.fasterxml.jackson.databind.*;
 import org.whispersystems.libsignal.SignalProtocolAddress;
 import org.whispersystems.libsignal.state.SessionRecord;


@@ -20,7 +19,7 @@ class JsonSessionStore implements SessionStore {
 
     }
 
 
     }
 

-    public void addSessions(Map<SignalProtocolAddress, byte[]> sessions) {
+    private void addSessions(Map<SignalProtocolAddress, byte[]> sessions) {

         this.sessions.putAll(sessions);
     }
 
         this.sessions.putAll(sessions);
     }
 

diff --git a/src/main/java/org/asamk/signal/storage/protocol/JsonSignalProtocolStore.java b/src/main/java/org/asamk/signal/storage/protocol/JsonSignalProtocolStore.java
index 1252243286e1662be0b28dd75d190c356dd4af32..6fcb052b90e1b47a90074a0c31c199fdf34df4df 100644 (file)
--- a/src/main/java/org/asamk/signal/storage/protocol/JsonSignalProtocolStore.java
+++ b/src/main/java/org/asamk/signal/storage/protocol/JsonSignalProtocolStore.java
@@ -21,22 +21,22 @@ public class JsonSignalProtocolStore implements SignalProtocolStore {
     @JsonProperty("preKeys")
     @JsonDeserialize(using = JsonPreKeyStore.JsonPreKeyStoreDeserializer.class)
     @JsonSerialize(using = JsonPreKeyStore.JsonPreKeyStoreSerializer.class)
     @JsonProperty("preKeys")
     @JsonDeserialize(using = JsonPreKeyStore.JsonPreKeyStoreDeserializer.class)
     @JsonSerialize(using = JsonPreKeyStore.JsonPreKeyStoreSerializer.class)

-    protected JsonPreKeyStore preKeyStore;
+    private JsonPreKeyStore preKeyStore;

 
     @JsonProperty("sessionStore")
     @JsonDeserialize(using = JsonSessionStore.JsonSessionStoreDeserializer.class)
     @JsonSerialize(using = JsonSessionStore.JsonPreKeyStoreSerializer.class)
 
     @JsonProperty("sessionStore")
     @JsonDeserialize(using = JsonSessionStore.JsonSessionStoreDeserializer.class)
     @JsonSerialize(using = JsonSessionStore.JsonPreKeyStoreSerializer.class)

-    protected JsonSessionStore sessionStore;
+    private JsonSessionStore sessionStore;

 
     @JsonProperty("signedPreKeyStore")
     @JsonDeserialize(using = JsonSignedPreKeyStore.JsonSignedPreKeyStoreDeserializer.class)
     @JsonSerialize(using = JsonSignedPreKeyStore.JsonSignedPreKeyStoreSerializer.class)
 
     @JsonProperty("signedPreKeyStore")
     @JsonDeserialize(using = JsonSignedPreKeyStore.JsonSignedPreKeyStoreDeserializer.class)
     @JsonSerialize(using = JsonSignedPreKeyStore.JsonSignedPreKeyStoreSerializer.class)

-    protected JsonSignedPreKeyStore signedPreKeyStore;
+    private JsonSignedPreKeyStore signedPreKeyStore;

 
     @JsonProperty("identityKeyStore")
     @JsonDeserialize(using = JsonIdentityKeyStore.JsonIdentityKeyStoreDeserializer.class)
     @JsonSerialize(using = JsonIdentityKeyStore.JsonIdentityKeyStoreSerializer.class)
 
     @JsonProperty("identityKeyStore")
     @JsonDeserialize(using = JsonIdentityKeyStore.JsonIdentityKeyStoreDeserializer.class)
     @JsonSerialize(using = JsonIdentityKeyStore.JsonIdentityKeyStoreSerializer.class)

-    protected JsonIdentityKeyStore identityKeyStore;
+    private JsonIdentityKeyStore identityKeyStore;

 
     public JsonSignalProtocolStore() {
     }
 
     public JsonSignalProtocolStore() {
     }

diff --git a/src/main/java/org/asamk/signal/storage/protocol/JsonSignedPreKeyStore.java b/src/main/java/org/asamk/signal/storage/protocol/JsonSignedPreKeyStore.java
index defd7f93f093725d1583a221825b944096e7f3cf..c8f4c3cc29a016422fd159c6820edc02b54ab84f 100644 (file)
--- a/src/main/java/org/asamk/signal/storage/protocol/JsonSignedPreKeyStore.java
+++ b/src/main/java/org/asamk/signal/storage/protocol/JsonSignedPreKeyStore.java
@@ -2,7 +2,6 @@ package org.asamk.signal.storage.protocol;
 
 import com.fasterxml.jackson.core.JsonGenerator;
 import com.fasterxml.jackson.core.JsonParser;
 
 import com.fasterxml.jackson.core.JsonGenerator;
 import com.fasterxml.jackson.core.JsonParser;

-import com.fasterxml.jackson.core.JsonProcessingException;

 import com.fasterxml.jackson.databind.*;
 import org.whispersystems.libsignal.InvalidKeyIdException;
 import org.whispersystems.libsignal.state.SignedPreKeyRecord;
 import com.fasterxml.jackson.databind.*;
 import org.whispersystems.libsignal.InvalidKeyIdException;
 import org.whispersystems.libsignal.state.SignedPreKeyRecord;


@@ -23,7 +22,7 @@ class JsonSignedPreKeyStore implements SignedPreKeyStore {
 
     }
 
 
     }
 

-    public void addSignedPreKeys(Map<Integer, byte[]> preKeys) {
+    private void addSignedPreKeys(Map<Integer, byte[]> preKeys) {

         store.putAll(preKeys);
     }
 
         store.putAll(preKeys);
     }
 

diff --git a/src/main/java/org/asamk/signal/storage/threads/JsonThreadStore.java b/src/main/java/org/asamk/signal/storage/threads/JsonThreadStore.java
index a9ce6fb651d0cc5400b29bce96dd64bd7331f40c..9ee613b8f1f62cc1865007bb151251960a4bd54b 100644 (file)
--- a/src/main/java/org/asamk/signal/storage/threads/JsonThreadStore.java
+++ b/src/main/java/org/asamk/signal/storage/threads/JsonThreadStore.java
@@ -34,7 +34,7 @@ public class JsonThreadStore {
         return new ArrayList<>(threads.values());
     }
 
         return new ArrayList<>(threads.values());
     }
 

-    public static class MapToListSerializer extends JsonSerializer<Map<?, ?>> {
+    private static class MapToListSerializer extends JsonSerializer<Map<?, ?>> {

 
         @Override
         public void serialize(final Map<?, ?> value, final JsonGenerator jgen, final SerializerProvider provider) throws IOException {
 
         @Override
         public void serialize(final Map<?, ?> value, final JsonGenerator jgen, final SerializerProvider provider) throws IOException {


@@ -42,7 +42,7 @@ public class JsonThreadStore {
         }
     }
 
         }
     }
 

-    public static class ThreadsDeserializer extends JsonDeserializer<Map<String, ThreadInfo>> {
+    private static class ThreadsDeserializer extends JsonDeserializer<Map<String, ThreadInfo>> {

 
         @Override
         public Map<String, ThreadInfo> deserialize(JsonParser jsonParser, DeserializationContext deserializationContext) throws IOException {
 
         @Override
         public Map<String, ThreadInfo> deserialize(JsonParser jsonParser, DeserializationContext deserializationContext) throws IOException {