Variable 'e' is no longer parsed properly

[keyneom]

A change was recently made to add support for scientific notation. I believe in doing so we lost the ability to use the built-in variable e.

Attempts to use any formula using the variable e as previously done now fail. e.g.

var engine = new CalculationEngine(CultureInfo.InvariantCulture, ExecutionMode.Compiled);
var result = engine.Calculate("e");

When using more complex functions (1 * e) you receive a rather unclear error message:

There is a syntax issue for the operation \"*\" at position 2. The number of arguments does not match with what is expected.

Reverting the following line change seems to avoid the issue, but I am sure will also prevent the scientific notation support from functioning properly:

Jace/Jace/Tokenizer/TokenReader.cs

Line 224 in 25b8d70

return character == decimalSeparator || (character >= '0' && character <= '9') || (isFormulaSubPart && isFirstCharacter && character == '-') || character == 'e' || character == 'E';

[keyneom]

@pieterderycke Was this an intended change? There are some performance changes recently released I am anxious to try out but the above issue is preventing us from doing so.

[pieterderycke]

@keyneom Thanks for letting me know. I will look to fix the bug and to release a bug fix release

[pieterderycke]

I have released a Jace 0.9.2 to fix this bug

[keyneom]

Amazingly fast!! Thank you so much!!