Skip to content

resolve: Avoid emitting redundant path resolution errors - #32789

Merged
bors merged 3 commits into
rust-lang:masterfrom
jseyfried:fix_duplicate_resolve_errors
Apr 7, 2016
Merged

resolve: Avoid emitting redundant path resolution errors#32789
bors merged 3 commits into
rust-lang:masterfrom
jseyfried:fix_duplicate_resolve_errors

Conversation

@jseyfried

Copy link
Copy Markdown
Contributor

This PR avoids emitting redundant path resolution errors in resolve (fixes#32760).

r? @eddyb

Comment threadsrc/test/compile-fail/issue-3521.rs Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the const evaluation error triggering for DefError? Could we suppress that, too?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe so -- I'll look into it.

@eddyb

eddyb commented Apr 7, 2016

Copy link
Copy Markdown
Contributor

r=me with tidy errors fixed.

@jseyfried

Copy link
Copy Markdown
ContributorAuthor

@eddyb Thanks for the quick review!
@bors r=eddyb

@bors

bors commented Apr 7, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit d3affc6 has been approved by eddyb

@jseyfried
jseyfriedforce-pushed the fix_duplicate_resolve_errors branch from d3affc6 to 07dac97CompareApril 7, 2016 05:50
@jseyfried

Copy link
Copy Markdown
ContributorAuthor

@bors r=eddyb

@bors

bors commented Apr 7, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit 07dac97 has been approved by eddyb

@ManishearthManishearth mentioned this pull request Apr 7, 2016
Manishearth added a commit to Manishearth/rust that referenced this pull request Apr 7, 2016
…ors, r=eddyb
resolve: Avoid emitting redundant path resolution errors
This PR avoids emitting redundant path resolution errors in `resolve` (fixesrust-lang#32760).
r? @eddyb
bors added a commit that referenced this pull request Apr 7, 2016
Rollup of 7 pull requests
- Successful merges: #32674, #32699, #32711, #32745, #32748, #32757, #32789
- Failed merges:
@bors
bors merged commit 07dac97 into rust-lang:masterApr 7, 2016
Sign up for freeto 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.

Duplicate path resolution errors in resolve

3 participants

@jseyfried@eddyb@bors