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

[1.38] Regression: expected module, found unresolved item crate::mod #63891

Closed
petrochenkov opened this issue Aug 25, 2019 · 1 comment
Closed
Labels
regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@petrochenkov
Copy link
Contributor

#63628 (comment)

expected module, found unresolved item crate::mod

./gh/xi-frontend/xi-term/beta-2019-08-13.txt:[INFO] [stderr] error[E0577]: expected module, found unresolved item `crate::mod`

This is the most hilarious case so far.
Apparently due to some recovery going wrong paths like {crate,self,super}::r#keyword silently resolve to Res::Err without reporting any error (I've made an issue for this - #63882).

Code in xrl-0.0.7 looked like this

pub(in crate::r#mod) fn new() { ... }

and it broke when visibility resolution switched to a different algorithm ("early resolution") in #63400, which doesn't have this bug.
I'd expect this issue to be discoverable by some kind of fuzzing, but I'd never expect it to be encountered by real code!

So, the regression is a bugfix and this bug needs to be fixed in other contexts as well, #63882 is the tracking issue for that.

@petrochenkov petrochenkov added regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Aug 25, 2019
@Mark-Simulacrum Mark-Simulacrum added this to the 1.38 milestone Aug 30, 2019
@nikomatsakis
Copy link
Contributor

check-in from compiler triage: this is the result of a bug-fix and is considered expected breakage. Closing.

@Centril Centril removed this from the 1.38 milestone Sep 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

4 participants