Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix issue #144 if/else if/else expression with trailing blanks does not evaluate properly #356

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

vboulaye
Copy link

@vboulaye vboulaye commented Mar 5, 2024

A few years later, I just stumbled across the same error.
But this time I think I know how to fix it!

While debugging I found out that the problem disappear on the second compilation of the same expression.
And I saw that the end property is not updated properly when setting the expression as a String and removing trailing blanks in AbstractParser.setExpression(String expression)

So here is my attempt to fix this issue.

@ScottCUSA
Copy link

ScottCUSA commented Mar 22, 2024

This might also fix #342 or at least the issues may be related.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants