Skip to content

feat: add signed type and const array support to SDL parser#42

Closed
edrthomas wants to merge 1 commit intoMPEGGroup:devfrom
edrthomas:dev
Closed

feat: add signed type and const array support to SDL parser#42
edrthomas wants to merge 1 commit intoMPEGGroup:devfrom
edrthomas:dev

Conversation

@edrthomas
Copy link

  • Add signed keyword support to ElementaryType definition
  • Add const qualifier support to ArrayDefinition rule (was missing)
  • Update AST mapping files for new token types

- Add `signed` keyword support to ElementaryType definition
- Add `const` qualifier support to ArrayDefinition rule (was missing)
- Update AST mapping files for new token types
@vectronic
Copy link
Collaborator

@edrthomas as discussed:

  • signed int is an alias for int and I think is unnecessary in SDL.
  • const qualifier for arrays is not defined in edition 1 of SDL. However in the current draft of edition 2 of SDL, const arrays are defined for computer array variables. Note that the use of const for parsed variables in general is (I hope) soon to be removed, so the concept of a const parsed array will not be required.

@edrthomas
Copy link
Author

edrthomas commented Mar 11, 2026

As discussed, let me close this PR for now.

@edrthomas edrthomas closed this Mar 11, 2026
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.

2 participants