Revert "Fix a memory leak in ExpressionParser.cpp" This reverts commit cce809ac90cc195f0c88a6cc788b3ff7359083a5. The code was actually correct: "expr" is never allocated when an error is returned. This means when the expression parser fails, deleting "expr" means deleting an uninitialized pointer.