Skip to content

emacs: Don't overwrite font lock for attributes, misc fixes #14990

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

Merged
merged 3 commits into from
Jun 18, 2014

Conversation

tomjakubowski
Copy link
Contributor

This addresses the font lock regression introduced by the earlier pull
request #14818 - attributes are no longer be highligted inside of comments
and strings.

Also add some font lock test infrastructure and some tests for attribute
font locking and fix some minor nits.

This addresses the font lock regression introduced by the earlier pull
request rust-lang#14818 - attributes are no longer be highligted inside of comments
and strings.

Also add some font lock test infrastructure and some tests for attribute
font locking.
@tomjakubowski
Copy link
Contributor Author

BTW, a couple of the indent test cases were failing for me before I made any changes.

@tomjakubowski
Copy link
Contributor Author

cc @pnkfelix

bors added a commit that referenced this pull request Jun 18, 2014
…ht, r=pnkfelix

This addresses the font lock regression introduced by the earlier pull
request #14818 - attributes are no longer be highligted inside of comments
and strings.

Also add some font lock test infrastructure and some tests for attribute
font locking and fix some minor nits.
@bors bors closed this Jun 18, 2014
@bors bors merged commit 23a7d24 into rust-lang:master Jun 18, 2014
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 17, 2023
Map our diagnostics to rustc and clippy's ones

And control their severity by lint attributes `#[allow]`, `#[deny]` and ... .

It doesn't work with proc macros and I would like to fix that before merge but I don't know how to do it.
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.

3 participants