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

change span_notes to notes in E0368/E0369 #39707

Merged
merged 1 commit into from
Feb 10, 2017

Conversation

durka
Copy link
Contributor

@durka durka commented Feb 9, 2017

Fixes #39650.

All the uses of span_note in these errors were reusing the same span as the original error, which causes unnecessary repetition.

For an example, see the changes to src/test/ui/span/issue-39018.stderr.

r? @jonathandturner

@sophiajt
Copy link
Contributor

sophiajt commented Feb 9, 2017

This looks good.

@bors r+

@bors
Copy link
Contributor

bors commented Feb 9, 2017

📌 Commit 79bf98d has been approved by jonathandturner

@durka durka force-pushed the parsimonious-span-note branch from 79bf98d to 9fffd14 Compare February 9, 2017 22:45
@durka
Copy link
Contributor Author

durka commented Feb 9, 2017

Tidy failure fixed.

@sophiajt
Copy link
Contributor

sophiajt commented Feb 9, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Feb 9, 2017

📌 Commit 9fffd14 has been approved by jonathandturner

frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 10, 2017
…athandturner

change span_notes to notes in E0368/E0369

Fixes rust-lang#39650.

All the uses of `span_note` in these errors were reusing the same span as the original error, which causes unnecessary repetition.

For an example, see the changes to [src/test/ui/span/issue-39018.stderr](https://github.com/rust-lang/rust/pull/39707/files?diff=unified#diff-46336f62958fdb34233db414cb9914a1R4).

r? @jonathandturner
bors added a commit that referenced this pull request Feb 10, 2017
Rollup of 6 pull requests

- Successful merges: #39587, #39674, #39693, #39700, #39705, #39707
- Failed merges:
@bors bors merged commit 9fffd14 into rust-lang:master Feb 10, 2017
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