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

Predefine QB64PE as $LET value #551

Closed
electric-socket opened this issue Oct 4, 2024 · 0 comments · Fixed by #558
Closed

Predefine QB64PE as $LET value #551

electric-socket opened this issue Oct 4, 2024 · 0 comments · Fixed by #558
Labels
enhancement New feature or request

Comments

@electric-socket
Copy link

electric-socket commented Oct 4, 2024

Is your feature request related to a problem? Please describe.
Because someone may be using features exclusive to QB64 Phoenix Edition, there should be a compile-time variable, i.e. a $LET variable QB64PE defined, just as one can check for a correct version of the compiler.

Describe the solution you'd like
Have QB64PE as a predefined $IF/$LET compile-time variable.

Describe alternatives you've considered
Short of the compiler choking on a construct when compiling, there's no way to be certain a particular code works with QB64PE than to have it crash[1], if, for example, QB64 level code is used in a main file but include files use QB64PE-specific features. Even checking version numbers might not be sufficient if the original QB64 has new feature updates and had a version number similar to QB64PE.

[1] "Crash" in this context means "IDE chokes and red flags it."

@electric-socket electric-socket added the enhancement New feature or request label Oct 4, 2024
RhoSigma-QB64 added a commit that referenced this issue Oct 15, 2024
- feature request, closes #551
@RhoSigma-QB64 RhoSigma-QB64 linked a pull request Oct 15, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant