Skip to content

Remove the span from ast::Lit.#101516

Closed
nnethercote wants to merge 1 commit intorust-lang:masterfrom
nnethercote:rm-span-from-ast-Lit
Closed

Remove the span from ast::Lit.#101516
nnethercote wants to merge 1 commit intorust-lang:masterfrom
nnethercote:rm-span-from-ast-Lit

Conversation

@nnethercote
Copy link
Contributor

In the most common case, the Lit is stored within an Expr, which records the same span.

For other cases, we can store the span next to the Lit.

r? @ghost

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants