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

Should ice2slice Use the New MetadataValidator? #3257

Open
InsertCreativityHere opened this issue Dec 11, 2024 · 1 comment
Open

Should ice2slice Use the New MetadataValidator? #3257

InsertCreativityHere opened this issue Dec 11, 2024 · 1 comment
Labels
metadata proposal slice compiler
Milestone
3.8.0

Comments

@InsertCreativityHere
Copy link
Member

Currently ice2slice performs no validation of metadata.
I think it would be preferable to still validate metadata, even if it is 'different' from the other Slice compilers.
We could also co-opt the MetadataValidator to issue warnings for metadata that we can't map in new Slice.
@InsertCreativityHere InsertCreativityHere added this to the 3.8.0 milestone Dec 11, 2024
@pepone
Copy link
Member

pepone commented Dec 17, 2024

How would you deal with language specific metadata, say for example cs:generic which we map to cs::type, or cs:namespace mapped to cs::namespace? I would be good if we can reuse the same validators from slice2cs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
metadata proposal slice compiler
Projects
None yet
Milestone
3.8.0
Development

No branches or pull requests
2 participants
@pepone @InsertCreativityHere