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

CTest uses syntex_syntax which doesn't support raw identifiers. #6

Open
ghost opened this issue Apr 30, 2020 · 2 comments
Open

CTest uses syntex_syntax which doesn't support raw identifiers. #6

ghost opened this issue Apr 30, 2020 · 2 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@ghost
Copy link

ghost commented Apr 30, 2020

CTest fails on code that uses raw identifiers like r#type. Currently libc translation of structs that have a type field in c are translated as type_ because of that.

If CTest would support raw identifiers we could accurately use type as a field name in rust too.

@JohnTitor
Copy link
Owner

JohnTitor commented Aug 21, 2020

I'd say it would be quite hard as syntex is quite old. So we should:

  1. Find the possible oldest version that supports the raw identifiers.
  2. Update syntex (note that we now use https://github.com/JohnTitor/garando instead to remove some outdated dependencies)
  3. Make sure that all the things aren't broken.

@JohnTitor
Copy link
Owner

Maybe we could use syn crate alternatively but it'd be a big change.

@JohnTitor JohnTitor linked a pull request May 11, 2022 that will close this issue
@JohnTitor JohnTitor added bug Something isn't working help wanted Extra attention is needed labels Aug 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant