Create a tree map from lowercase built-in function name to ID number

- BuiltinUsage#summaryText(String) doesn't require F.await() anymore, which leads to performance issue in Android UI thread.
axkr · Sep 30, 2023 · a136716 · a136716
1 parent a3fbf5d
commit a136716
Show file tree

Hide file tree

Showing 3 changed files with 453 additions and 58 deletions.
diff --git a/..._library/matheclipse-core/src/main/java/org/matheclipse/core/expression/BuiltinUsage.java b/..._library/matheclipse-core/src/main/java/org/matheclipse/core/expression/BuiltinUsage.java
@@ -2,7 +2,6 @@
 
 import java.util.concurrent.atomic.AtomicBoolean;
 import org.matheclipse.core.interfaces.IBuiltInSymbol;
-import org.matheclipse.core.interfaces.ISymbol;
 
 public class BuiltinUsage {
   private static final String[] USAGE = new String[ID.Zeta + 10];
@@ -607,12 +606,9 @@ public static synchronized void init() {
   }
 
   public static String summaryText(String symbolStr) {
-    if (!IS_INITIALIZED.get()) {
-      init();
-    }
-    ISymbol symbol = F.symbol(symbolStr);
-    if (symbol.isBuiltInSymbol()) {
-      String str = USAGE[symbol.ordinal()];
+    Integer ordinalId = ID.STRING_TO_ID_MAP.get(symbolStr);
+    if (ordinalId != null) {
+      String str = USAGE[ordinalId];
       return str != null ? str : "";
     }
     return "";