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.

Sign up for GitHub

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

Jump to bottom

Remove *begin, *end and *then #611

Closed
Daniel-Cortez opened this issue Nov 11, 2020 · 1 comment · May be fixed by #681
Closed

Remove *begin, *end and *then #611

Daniel-Cortez opened this issue Nov 11, 2020 · 1 comment · May be fixed by #681
Labels
area: syntax state: discuss type: enhancement

Comments

@Daniel-Cortez
Copy link
Contributor

Daniel-Cortez commented Nov 11, 2020 •
edited
Loading

Issue description:

  • These keywords don't have any unique meaning; they only duplicate the functionality of the existing C-style curly bracket syntax.
  • They can complicate diagnostics, causing counter-intuitive error messages, as shown in Bizzare error for invalid if #512.
  • They were already removed in Pawn 3.3, so we could do the same in this fork.

Minimal complete verifiable example (MCVE):

Workspace Information:

  • Compiler version: 3.10.10
  • Command line arguments provided (or sampctl version):
  • Operating System:
@Daniel-Cortez Daniel-Cortez mentioned this issue Sep 30, 2021
Open
4 tasks
@YashasSamaga YashasSamaga linked a pull request Oct 8, 2021 that will close this issue
Remove keywords *begin, *end and *then #681
Open
4 tasks
@Y-Less Y-Less closed this as completed Mar 31, 2022
@Daniel-Cortez
Copy link
Contributor Author

Daniel-Cortez commented Apr 1, 2022 •
edited
Loading

@Y-Less Why did you close this issue if the PR that's supposed to fix it is still not merged?
EDIT: I see, it seems that there's a build error in that PR (and I honestly don't remember having in on my PC); I'll see if I can fix it this weekend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area: syntax state: discuss type: enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove keywords *begin, *end and *then Daniel-Cortez/pawn-3.10
3 participants
@Y-Less @Daniel-Cortez @YashasSamaga