Fix #555

[st0012]

This closes #555

[codecov]

Codecov Report

Merging #683 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #683      +/-   ##
==========================================
+ Coverage   80.24%   80.25%   +<.01%     
==========================================
  Files          54       54              
  Lines        7402     7403       +1     
==========================================
+ Hits         5940     5941       +1     
  Misses       1237     1237              
  Partials      225      225

Impacted Files	Coverage Δ
compiler/parser/parser.go	94.11% <100%> (+0.04%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 444cb91...a7a0a5d. Read the comment docs.

[hachi8833]

Sorry for the delay: I'd check this 🙇

[hachi8833]

Thank you very much for fixing the issue!

I prepared exhaustive tests for the issue, based on the old PR I once made.
And found some errors that need fixes.

How do I send the test to you?

[st0012]

You can leave them in comment or push to the branch

[hachi8833]

Pushed the updated branch.
You can see comments on errors in evaluation_test.go @st0012

[hachi8833]

Just to memorize the additional tests for the PR

555_test.txt

[st0012]

@hachi8833 can you give it a quick review and I'll just merge it

[hachi8833]

As far as I checked with the unary operators + and - on the fix, no wrong behavior has been found. Thank you very much!

(I was amazed the fix is very very small😳)

[hachi8833]

@st0012 I noticed that this PR also fixes #568 as far as I rebased this to current master locally.
(Rebasing is OK, but several tests need fixes)

[st0012]

@hachi8833 really? I can't verify that right now, you can close #568 if you feel confident about the fix. I'd say it's ok to close it if we don't see errors from examples on the top description.

[hachi8833]

I will verify if #568 is fixed after merging and close it. 👍

[hachi8833]

Strange... this time the issue on #568 still persists😓. Forget it.