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

Stable support #1

Merged
merged 7 commits into from
Nov 19, 2018
Merged

Stable support #1

merged 7 commits into from
Nov 19, 2018

Conversation

dtolnay
Copy link
Contributor

@dtolnay dtolnay commented Nov 18, 2018

This PR makes typed-html work with stable rustc as of version 1.30.0. Feel free to close if you prefer to wait for proc_macro_hygiene, although I believe that is quite a ways away from being stabilized.

@bodil
Copy link
Owner

bodil commented Nov 18, 2018

This is a tough one - I'm amazed that the error reporting is as good as this, because I couldn't figure out how to do this myself with the proc_macros2 stack - it's the reason I stuck with the native stuff. On the other hand, the errors aren't quite as good now, even though they're strictly adequate. Might be a price worth paying for stable support, let alone how much cleaner the code would be with the improved quote macro - I'll have to think about it for a little bit.

It's a very impressive PR in any case, thanks!

@dtolnay
Copy link
Contributor Author

dtolnay commented Nov 18, 2018

Which errors are you looking at that are not quite as good? I didn't get a chance to test all the error codepaths locally so it could be an omission (filed #2 to follow up on automated testing).

@bodil
Copy link
Owner

bodil commented Nov 19, 2018

The errors generated by the HTML parsers come out as two errors rather than an error with an attached note, but tbh that's mostly just a cosmetic issue.

Anyway, I've thought about it and decided it's definitely worth merging. Thanks. 😊

@dtolnay
Copy link
Contributor Author

dtolnay commented Nov 19, 2018

I filed #5 to follow up on the cosmetic issue.

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