Skip to content

vim: move 'move' to rustStorage #18789

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 1 commit into from
Nov 11, 2014
Merged

vim: move 'move' to rustStorage #18789

merged 1 commit into from
Nov 11, 2014

Conversation

cuviper
Copy link
Member

@cuviper cuviper commented Nov 8, 2014

It used to be in rustKeyword, until commit 5c75f21 removed it, and then #18782 restored it again. However, this is now a closure modifier, and I think moving it to rustStorage is more appropriate to highlight it similarly to mut, ref, and the & sigil.

@rust-highfive
Copy link
Contributor

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nikomatsakis (or someone else) soon.

@ftxqxd
Copy link
Contributor

ftxqxd commented Nov 10, 2014

#18782 seems to do this as well, although it adds move to rustKeyword instead.

bors added a commit that referenced this pull request Nov 10, 2014
It used to be in `rustKeyword`, but commit 5c75f21 removed it.  It was reintroduced as a keyword in 0.12.0 to indicate by-value closures, and I think `rustStorage` highlighting makes sense for that purpose.
@alexcrichton
Copy link
Member

Ah yes, I forgot about that, thanks @P1start! @cuviper are you ok with it just as a keyword for now (instead of the storage class)?

@cuviper
Copy link
Member Author

cuviper commented Nov 10, 2014

It's OK, but I think the storage class is better. :) I'll ask @netvl on the other issue.

@cuviper cuviper changed the title vim: add 'move' to rustStorage vim: move 'move' to rustStorage Nov 10, 2014
@cuviper
Copy link
Member Author

cuviper commented Nov 10, 2014

Ok, @netvl doesn't mind, so I've now rebased this as a change to his placement of move.

bors added a commit that referenced this pull request Nov 11, 2014
It used to be in `rustKeyword`, until commit 5c75f21 removed it, and then #18782 restored it again.  However, this is now a closure modifier, and I think moving it to `rustStorage` is more appropriate to highlight it similarly to `mut`, `ref`, and the `&` sigil.
@bors bors closed this Nov 11, 2014
@bors bors merged commit e94912a into rust-lang:master Nov 11, 2014
@cuviper cuviper deleted the vim-move branch November 11, 2014 22:31
lnicola pushed a commit to lnicola/rust that referenced this pull request Jan 7, 2025
fix: Fix invalid `-O` flag used by cfg discovery
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.

6 participants