3
Vote

Remove debugging call to improve performance

description

This is the line that I am referring to. I do not think it is appropriate in a library.

https://github.com/tyrone-sudeium/NCalc/blob/c141185099082663e5c7651e6b87823c9887feb3/Evaluant.Calculator/Expression.cs#L155

I believe that it should be removed

comments

beerwin wrote Jun 29, 2016 at 10:24 AM

I concur.

I'm using it in batch processing some data and it is very annoying, as it litters the Output window with lots of Expression retrieved from cache and Expression added to cache lines, making it very difficult to find my own output messages or error messages.

A property capable of turning these on or off would suffice (off by default).

Since there are no reliable means to filter the Output window, this is a must have.