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

Improve Ordinal Declaration Handling #55

Open
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

montymxb
Copy link
Contributor

Reworks ordinals a bit to allow:

  • multiple redundant define ordinal xyz (...) signed signed or define ordinal xyz (...) unsigned unsigned
  • mixed signed & unsigned attributes in any number, which pass in the 6.1 compiler w/out any issues
  • adding to that, this does add a warning here to the case when mixed attributes are detected
  • this also adds a warning when multiple precisions are detected, for as similar reason
  • test cases to enforce expected behavior

Signed-off-by: Benjamin Friedman Wilson <[email protected]>
Signed-off-by: Benjamin Friedman Wilson <[email protected]>
Copy link

PR Preview Action v1.6.0

🚀 View preview at
https://zowe.github.io/zowe-pli-language-support/pr/pr-55/

Built to branch previews at 2025-02-25 15:43 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant