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

Move tokens method from TranslationUnit to Cursor #1173

Merged
merged 1 commit into from Dec 3, 2017

Conversation

Projects
None yet
4 participants
@nox
Copy link
Contributor

nox commented Dec 3, 2017

This makes it easier to call it without passing around a TranslationUnit,
which will prove itself useful to parse __attribute__ nodes.

Move tokens method from TranslationUnit to Cursor
This makes it easier to call it without passing around a TranslationUnit,
which will prove itself useful to parse `__attribute__` nodes.
@highfive

This comment has been minimized.

Copy link
Collaborator

highfive commented Dec 3, 2017

warning Warning warning

  • These commits modify unsafe code. Please review it carefully!
@emilio

emilio approved these changes Dec 3, 2017

@emilio

This comment has been minimized.

Copy link
Collaborator

emilio commented Dec 3, 2017

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 3, 2017

📌 Commit d076b9d has been approved by emilio

bors-servo added a commit that referenced this pull request Dec 3, 2017

Auto merge of #1173 - nox:tokens, r=emilio
Move tokens method from TranslationUnit to Cursor

This makes it easier to call it without passing around a TranslationUnit,
which will prove itself useful to parse `__attribute__` nodes.
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 3, 2017

⌛️ Testing commit d076b9d with merge f08aa38...

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 3, 2017

💔 Test failed - status-travis

@nox

This comment has been minimized.

Copy link
Contributor Author

nox commented Dec 3, 2017

@bors-servo retry

bors-servo added a commit that referenced this pull request Dec 3, 2017

Auto merge of #1173 - nox:tokens, r=emilio
Move tokens method from TranslationUnit to Cursor

This makes it easier to call it without passing around a TranslationUnit,
which will prove itself useful to parse `__attribute__` nodes.
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 3, 2017

⌛️ Testing commit d076b9d with merge 74d1eaa...

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 3, 2017

💔 Test failed - status-travis

@nox

This comment has been minimized.

Copy link
Contributor Author

nox commented Dec 3, 2017

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 3, 2017

⌛️ Trying commit c631736 with merge 555216b...

bors-servo added a commit that referenced this pull request Dec 3, 2017

Auto merge of #1173 - nox:tokens, r=<try>
Move tokens method from TranslationUnit to Cursor

This makes it easier to call it without passing around a TranslationUnit,
which will prove itself useful to parse `__attribute__` nodes.

@nox nox force-pushed the nox:tokens branch 2 times, most recently from 6bbdbcb to d076b9d Dec 3, 2017

@emilio emilio merged commit 935a18e into rust-lang:master Dec 3, 2017

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
homu Test failed
Details
@emilio

This comment has been minimized.

Copy link
Collaborator

emilio commented Dec 3, 2017

Larsborsing because it's not clear what to do about rustfmt...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.