diff --git a/java/src/processing/mode/java/JavaEditor.java b/java/src/processing/mode/java/JavaEditor.java
index c45a29263..202f2c19a 100644
--- a/java/src/processing/mode/java/JavaEditor.java
+++ b/java/src/processing/mode/java/JavaEditor.java
@@ -2764,11 +2764,9 @@ public class JavaEditor extends Editor {
   //private int howManyInts(ArrayList<Handle> handles[])
   static private int howManyInts(List<List<Handle>> handles) {
     int count = 0;
-    //for (int i=0; i<handles.length; i++) {
     for (List<Handle> list : handles) {
-      //for (Handle n : handles[i]) {
       for (Handle n : list) {
-        if (n.type == "int" || n.type == "hex" || n.type == "webcolor") {
+        if ("int".equals(n.type) || "hex".equals(n.type) || "webcolor".equals(n.type)) {
           count++;
         }
       }
@@ -2780,11 +2778,9 @@ public class JavaEditor extends Editor {
   //private int howManyFloats(ArrayList<Handle> handles[])
   static private int howManyFloats(List<List<Handle>> handles) {
     int count = 0;
-    //for (int i=0; i<handles.length; i++) {
     for (List<Handle> list : handles) {
-      //for (Handle n : handles[i]) {
       for (Handle n : list) {
-        if (n.type == "float") {
+        if ("float".equals(n.type)) {
           count++;
         }
       }
diff --git a/todo.txt b/todo.txt
index cd83a648a..1314798aa 100755
--- a/todo.txt
+++ b/todo.txt
@@ -22,6 +22,8 @@ X update launch4j to 3.9
 X   https://sourceforge.net/projects/launch4j/files/launch4j-3/3.9/
 X exported application doesn't work with latest jre
 X   https://github.com/processing/processing/issues/4682
+X string comparison fix
+X   https://github.com/processing/processing/issues/4670
 
 gohai
 X Simplify font situation to make it possible to use vanilla JRE trees