TarCV / botc

Fork of Dusk's botc compiler *OPEN FOR ADOPTION*
Other
1 stars 0 forks source link

Fix flooding stack with unused returns for command calls #14

Closed TarCV closed 4 years ago

TarCV commented 4 years ago

Fixes #10

This is a quick fix. Proper fix should also keep note of current stack depth and assert it is empty when closing scope (or may be on semicolon).

TarCV commented 4 years ago

Actually, revisiting that, I don't think there are any other statement/expression that can leave unused values on stacks. So the fix as is should be enough. Also with soon coming updates to debotc, similar issues should be caught in CI.