Consolidate formula extraction - still checked twice...

Author: govert

Source/ExcelDna.IntelliSense.Host/MyFunctions.cs

@@ -28,7 +28,7 @@ public static object AnotherFunction( [Description("In and out")] object inout)
                 return inout;
             }
 
-            [ExcelFunction]
+            [ExcelFunction(Name ="A.Non.Descript.Function")]
             public static object ANonDescriptFunction(object inout)
             {
                 return inout;

Source/ExcelDna.IntelliSense/FormulaParser.cs

@@ -8,7 +8,7 @@ static class FormulaParser
         // TODO: This needs a proper implementation, considering subformulae
         internal static bool TryGetFormulaInfo(string formulaPrefix, out string functionName, out int currentArgIndex)
         {
-            var match = Regex.Match(formulaPrefix, @"^=(?<functionName>\w*)\(");
+            var match = Regex.Match(formulaPrefix, @"^=(?<functionName>(\w|.)*)\(");
             if (match.Success)
             {
                 functionName = match.Groups["functionName"].Value;

Source/ExcelDna.IntelliSense/IntelliSenseDisplay.cs

@@ -102,12 +102,13 @@ bool ShouldProcessFormulaEditTextChange(string formulaPrefix)
             if (_argumentsToolTip?.Visible == true)
                 return true;
 
-            // TODO: Consolidate the check here with the FormulaMonitor
-            var match = Regex.Match(formulaPrefix, @"^=(?<functionName>\w*)\(");
-            if (match.Success)
+            // TODO: Why do this twice....?
+            string functionName;
+            int currentArgIndex;
+            if (FormulaParser.TryGetFormulaInfo(formulaPrefix, out functionName, out currentArgIndex))
             {
-                string functionName = match.Groups["functionName"].Value;
-                return _functionInfoMap.ContainsKey(functionName);
+                if (_functionInfoMap.ContainsKey(functionName))
+                    return true;
             }
             // Not interested...
             Debug.Print($"Not processing formula {formulaPrefix}");