Changeset 15934 in josm for trunk/src/org/openstreetmap/josm/gui

Timestamp:

2020-02-26T01:52:17+01:00 (5 years ago)

Author:

simon04

Message:

see #18749 - Intern strings to reduce memory footprint

Location:

trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss

Files:

• ConditionFactory.java (modified) (2 diffs)
• LiteralExpression.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/ConditionFactory.java

@@ -259 +259 @@
          */
         public SimpleKeyValueCondition(String k, String v) {
-            this.k = k;
-            this.v = v;
+            this.k = k.intern();
+            this.v = v.intern();
         }
 
@@ -311 +311 @@
          */
         public KeyValueCondition(String k, String v, Op op, boolean considerValAsKey) {
-            this.k = k;
-            this.v = v;
+            this.k = k.intern();
+            this.v = v.intern();
             this.op = op;
             this.considerValAsKey = considerValAsKey;

trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/LiteralExpression.java

@@ -20 +20 @@
     public LiteralExpression(Object literal) {
         CheckParameterUtil.ensureParameterNotNull(literal);
-        this.literal = literal;
+        this.literal = literal instanceof String ? ((String) literal).intern() : literal;
     }