Open sadqiang opened 5 years ago
It would also be more consistent with the rest of the API.
Technically, if we'd change the argument to expect a SymbolicExpression, the former notation with a string would still work due to the implicit cast from string to the expression. We'd need to verify it also works with the params array overload though.
I think hardcoding "x" more than once should be avoided because it is error prone. It potentially causes runtime errors.
My suggestion is as follows:
Any comments are welcome. Thank you.