Skip to content

Conversation

@nox
Copy link
Contributor

@nox 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.

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

highfive commented Dec 3, 2017

warning Warning warning

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

@emilio
Copy link
Contributor

emilio commented Dec 3, 2017

@bors-servo r+

@bors-servo
Copy link

📌 Commit d076b9d has been approved by emilio

bors-servo pushed a commit that referenced this pull request Dec 3, 2017
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
Copy link

⌛ Testing commit d076b9d with merge f08aa38...

@bors-servo
Copy link

💔 Test failed - status-travis

@nox
Copy link
Contributor Author

nox commented Dec 3, 2017

@bors-servo retry

bors-servo pushed a commit that referenced this pull request Dec 3, 2017
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
Copy link

⌛ Testing commit d076b9d with merge 74d1eaa...

@bors-servo
Copy link

💔 Test failed - status-travis

@nox
Copy link
Contributor Author

nox commented Dec 3, 2017

@bors-servo try

@bors-servo
Copy link

⌛ Trying commit c631736 with merge 555216b...

bors-servo pushed a commit that referenced this pull request Dec 3, 2017
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 tokens branch 2 times, most recently from 6bbdbcb to d076b9d Compare December 3, 2017 19:11
@emilio emilio merged commit 935a18e into rust-lang:master Dec 3, 2017
@emilio
Copy link
Contributor

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants