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

Make ItemContext available for better diagnositcs #65902

Merged
merged 1 commit into from
Nov 1, 2019

Conversation

gilescope
Copy link
Contributor

@gilescope gilescope commented Oct 28, 2019

Fix #62570

@rust-highfive
Copy link
Collaborator

r? @cramertj

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 28, 2019
@gilescope gilescope changed the title WIP: Make ItemContext available for better diagnositcs. WIP: Make ItemContext available for better diagnositcs. Fixes #62570 Oct 28, 2019
@gilescope
Copy link
Contributor Author

Fixes #62570

@Centril
Copy link
Contributor

Centril commented Oct 28, 2019

r? @estebank

@rust-highfive rust-highfive assigned estebank and unassigned cramertj Oct 28, 2019
src/librustc/traits/error_reporting.rs Outdated Show resolved Hide resolved
src/librustc/traits/error_reporting.rs Outdated Show resolved Hide resolved
@rust-highfive

This comment has been minimized.

src/librustc/traits/error_reporting.rs Outdated Show resolved Hide resolved
src/librustc/traits/error_reporting.rs Outdated Show resolved Hide resolved
src/librustc/traits/error_reporting.rs Outdated Show resolved Hide resolved
@gilescope gilescope force-pushed the issue62570 branch 2 times, most recently from 803f1da to 8c5f0f6 Compare October 30, 2019 00:07
@gilescope gilescope changed the title WIP: Make ItemContext available for better diagnositcs. Fixes #62570 Make ItemContext available for better diagnositcs. Fixes #62570 Oct 30, 2019
@estebank
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Oct 31, 2019

📌 Commit d7869ec has been approved by estebank

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 31, 2019
@estebank estebank changed the title Make ItemContext available for better diagnositcs. Fixes #62570 Make ItemContext available for better diagnositcs Oct 31, 2019
tmandry added a commit to tmandry/rust that referenced this pull request Nov 1, 2019
Make ItemContext available for better diagnositcs

Fix rust-lang#62570
bors added a commit that referenced this pull request Nov 1, 2019
Rollup of 16 pull requests

Successful merges:

 - #65112 (Add lint and tests for unnecessary parens around types)
 - #65470 (Don't hide ICEs from previous incremental compiles)
 - #65471 (Add long error explanation for E0578)
 - #65857 (rustdoc: Resolve module-level doc references more locally)
 - #65902 (Make ItemContext available for better diagnositcs)
 - #65914 (Use structured suggestion for unnecessary bounds in type aliases)
 - #65946 (Make `promote_consts` emit the errors when required promotion fails)
 - #65960 (doc: reword iter module example and mention other methods)
 - #65963 (update submodules to rust-lang)
 - #65972 (Fix libunwind build: Define __LITTLE_ENDIAN__ for LE targets)
 - #65977 (Fix incorrect diagnostics for expected type in E0271 with an associated type)
 - #65995 (Add error code E0743 for "C-variadic has been used on a non-foreign function")
 - #65997 (Fix outdated rustdoc of Once::init_locking function)
 - #66002 (Stabilize float_to_from_bytes feature)
 - #66005 (vxWorks: remove code related unix socket)
 - #66018 (Revert PR 64324: dylibs export generics again (for now))

Failed merges:

r? @ghost
@bors bors merged commit d7869ec into rust-lang:master Nov 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Async blocks are leaky about wording wrt. ? in diagnostics
6 participants