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

Multiply included files should not be supported in the same definition #554

Closed
reid-spencer opened this issue Jan 8, 2024 · 0 comments · Fixed by #556
Closed

Multiply included files should not be supported in the same definition #554

reid-spencer opened this issue Jan 8, 2024 · 0 comments · Fixed by #556
Labels
bug Something isn't working
Milestone
1.0 - First Full ...

Comments

@reid-spencer
Copy link
Contributor

If you have the same include file in a single definition, it is going to duplicate the names in that definition which is not permitted. Such include scenarios should be detected and errors generated. Names in definitions must be unique.
@reid-spencer reid-spencer added the bug Something isn't working label Jan 8, 2024
@reid-spencer reid-spencer added this to the 1.0 - First Full Release milestone Jan 8, 2024
@reid-spencer reid-spencer linked a pull request Jan 12, 2024 that will close this issue
Implement 554 - warn about duplicate includes #556
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
riddl
Status: Done
Milestone
1.0 - First Full Release
Development

Successfully merging a pull request may close this issue.

Implement 554 - warn about duplicate includes ossuminc/riddl
1 participant
@reid-spencer