only:final

java_console/io/src/main/java/com/rusefi/binaryprotocol/BinaryProtocol.java

@@ -53,7 +53,7 @@ public class BinaryProtocol {
      * This properly allows to switch to non-CRC32 mode
      * todo: finish this feature, assuming we even need it.
      */
-    public static boolean PLAIN_PROTOCOL = Boolean.getBoolean(USE_PLAIN_PROTOCOL_PROPERTY);
+    public static final boolean PLAIN_PROTOCOL = Boolean.getBoolean(USE_PLAIN_PROTOCOL_PROPERTY);
 
     private final LinkManager linkManager;
     private final IoStream stream;
@@ -105,7 +105,7 @@ public class BinaryProtocol {
 
     public boolean isClosed;
 
-    public CommunicationLoggingListener communicationLoggingListener;
+    public final CommunicationLoggingListener communicationLoggingListener;
 
     public BinaryProtocol(LinkManager linkManager, IoStream stream) {
         this.linkManager = linkManager;

java_console/io/src/main/java/com/rusefi/io/ConnectionStatusLogic.java

@@ -27,7 +27,7 @@ public class ConnectionStatusLogic {
         }
     }
 
-    public static ConnectionStatusLogic INSTANCE = new ConnectionStatusLogic();
+    public static final ConnectionStatusLogic INSTANCE = new ConnectionStatusLogic();
     private final List<Listener> listeners = new CopyOnWriteArrayList<>();
 
     private ConnectionStatusLogic() {

java_console/io/src/main/java/com/rusefi/io/LinkManager.java

@@ -37,9 +37,9 @@ public class LinkManager implements Closeable {
     public static final String SOCKET_CAN = "SocketCAN";
 
     @NotNull
-    public static LogLevel LOG_LEVEL = LogLevel.INFO;
+    public static final LogLevel LOG_LEVEL = LogLevel.INFO;
 
-    public static LinkDecoder ENCODER = new LinkDecoder() {
+    public static final LinkDecoder ENCODER = new LinkDecoder() {
         @Override
         public String unpack(String packedLine) {
             return packedLine;

java_console/io/src/main/java/com/rusefi/io/serial/SerialIoStream.java

@@ -17,7 +17,7 @@ import java.io.IOException;
 import static com.devexperts.logging.Logging.getLogging;
 
 public class SerialIoStream extends AbstractIoStream {
-    static Logging log = getLogging(SerialIoStream.class);
+    static final Logging log = getLogging(SerialIoStream.class);
 
     @Nullable // null in case of port open error, for instance lack of permissions on Unix
     protected final SerialPort sp;

java_console/io/src/main/java/com/rusefi/io/tcp/BinaryProtocolServer.java

@@ -55,7 +55,7 @@ public class BinaryProtocolServer {
         log.configureDebugEnabled(false);
     }
 
-    public AtomicInteger unknownCommands = new AtomicInteger();
+    public final AtomicInteger unknownCommands = new AtomicInteger();
 
     public static final ServerSocketFunction SECURE_SOCKET_FACTORY = rusEFISSLContext::getSSLServerSocket;
 

java_console/io/src/test/java/com/rusefi/binaryprotocol/test/TcpServerSandbox.java

@@ -74,7 +74,7 @@ public class TcpServerSandbox {
     static class EcuState {
         private final byte[] outputs = new byte[Fields.TS_TOTAL_OUTPUT_SIZE];
 
-        long startUpTime = System.currentTimeMillis();
+        final long startUpTime = System.currentTimeMillis();
 
         public void onCommand() {
             int seconds = (int) ((System.currentTimeMillis() - startUpTime) / 1000);

java_console/models/src/main/java/com/rusefi/AverageAngles.java

@@ -15,8 +15,8 @@ import java.util.TreeMap;
  * 2/15/2015
  */
 public class AverageAngles {
-    static String PRIMARY = "T_PRIMARY";
+    static final String PRIMARY = "T_PRIMARY";
-    static String SECONDARY = "T_SECONDARY";
+    static final String SECONDARY = "T_SECONDARY";
     private int count;
 
     enum trigger_event_e {
@@ -34,7 +34,7 @@ public class AverageAngles {
 
     private static final int MAX_RPM_CHANGE = 20;
     private int rpmAtPrevChart;
-    Map<Integer, List<AngleEvent>> angleData = new TreeMap<>();
+    final Map<Integer, List<AngleEvent>> angleData = new TreeMap<>();
 
     public AverageAngles() {
         clear();

java_console/ui/src/main/java/com/rusefi/ui/UIContext.java

@@ -13,7 +13,7 @@ public class UIContext {
     public SensorLogger sensorLogger = new SensorLogger(this);
     public GaugesPanel.DetachedRepository DetachedRepositoryINSTANCE = new GaugesPanel.DetachedRepository(this);
 
-    public SensorSnifferCentral sensorSnifferCentral = new SensorSnifferCentral(linkManager);
+    public final SensorSnifferCentral sensorSnifferCentral = new SensorSnifferCentral(linkManager);
 
     @NotNull
     public LinkManager getLinkManager() {

java_console/ui/src/main/java/com/rusefi/ui/console/MainFrame.java

@@ -54,7 +54,7 @@ public class MainFrame {
         }
     };
 
-    public ConnectionFailedListener listener;
+    public final ConnectionFailedListener listener;
 
     public MainFrame(ConsoleUI consoleUI, TabbedPanel tabbedPane) {
         this.consoleUI = Objects.requireNonNull(consoleUI);

java_console/ui/src/main/java/com/rusefi/ui/lua/DocumentSizeFilter.java

@@ -41,7 +41,7 @@ import javax.swing.text.DocumentFilter;
 import java.awt.*;
 
 public class DocumentSizeFilter extends DocumentFilter {
-    int maxCharacters;
+    final int maxCharacters;
 
     public DocumentSizeFilter(int maxChars) {
         maxCharacters = maxChars;

java_tools/configuration_definition_base/src/main/java/com/rusefi/output/FieldIterator.java

@@ -11,7 +11,7 @@ import java.util.List;
  */
 public class FieldIterator {
     private final List<ConfigField> fields;
-    BitState bitState = new BitState();
+    final BitState bitState = new BitState();
     private ConfigField prev = ConfigField.VOID;
     ConfigField next;
     ConfigField cf;

java_tools/enum_to_string/src/main/java/com/rusefi/VariableRegistry.java

@@ -40,7 +40,7 @@ public class VariableRegistry {
     private final Pattern VAR = Pattern.compile("(@@(.*?)@@)");
     private final Pattern VAR_REMOVE_QUOTE = Pattern.compile("(@#(.*?)#@)");
 
-    public Map<String, Integer> intValues = new HashMap<>();
+    public final Map<String, Integer> intValues = new HashMap<>();
 
     private final Map<String, String> cAllDefinitions = new TreeMap<>(String.CASE_INSENSITIVE_ORDER);
     // todo: move thid logic to JavaFieldsConsumer since that's the consumer?