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

Remove then/begin/end tokens. #161

Merged
merged 2 commits into from Nov 7, 2017

Conversation

@Fyren
Copy link
Contributor

Fyren commented Nov 7, 2017

Currently, you can use *begin and *end to enclose compound statements in a lot of places as well as use *then to not require parentheses around the test for if statements. I have no idea why they have asterisks in the first place. Perhaps a lazy attempt to remove the syntax?

This patch additionally removes the ability to do while a == 1 do stmt();.

@dvander

This comment has been minimized.

Copy link
Member

dvander commented Nov 7, 2017

🚢 🚢 🚢 🚢 🚢 🚢 🚢 🚢 🚢 🚢 🚢 🚢 🚢 🚢 🚢 🚢 🚢 🚢 🚢

@dvander

This comment has been minimized.

Copy link
Member

dvander commented Nov 7, 2017

I'm pretty sure that was indeed a lazy attempt. I vaguely recall thinking the lexer would reject * because it would match earlier as just a '*', but of course that's not true.

@Fyren Fyren merged commit 00dc171 into master Nov 7, 2017
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Fyren Fyren deleted the murder-then-begin-end branch Nov 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.