Skip to content

Importing a nonexistent item does not cause compilation to fail #2937

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.

Sign up for GitHub

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

Jump to bottom
Closed
bstrie opened this issue Jul 17, 2012 · 1 comment
Closed

Importing a nonexistent item does not cause compilation to fail #2937

bstrie opened this issue Jul 17, 2012 · 1 comment
Labels
A-resolve Area: Name/path resolution done by `rustc_resolve` specifically
Milestone
0.5

Comments

@bstrie
Copy link
Contributor

bstrie commented Jul 17, 2012

Here's the test case from #2123. It used to just crash with a poor error message, but now it doesn't fail at all despite the lack of anything named m::f.

import x = m::f;

mod m {
}

fn main() {
}
@ghost ghost assigned catamorphism Jul 17, 2012
@catamorphism
Copy link
Contributor

I'll look at this.

matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Oct 11, 2018
@matthiaskrgr
submodules: update clippy from 32b1d1f to 9d33731
89e824a 
Changes:

Remove now-useless `allow(unknown_lints)`
Stabilize tool lints
Use `impl Iterator` in arg position in clippy_dev
Fix fn_to_numeric_cast_with_truncation suppression
Limit commutative assign op lint to primitive types
Clarify code
Fix rust-lang#2937
Fix cast_possible_wrap and cast_sign_loss warnings
Fix cast_possible_truncation warnings
Fixes rust-lang#2925 cmp_owned false positive
if_let_redundant_pattern_matching: use Span.to() instead of Span.with_hi() to fix crash.
Improve diagnostics in case of lifetime elision (closes rust-lang#3284)
Fix items_after_statements for `const`s
Fix items_after_statements for sub-functions
Fix items_after_statements for `use` statements
Don't suggest cloned() for map Box deref
Fix excessive_precision false positive
Fix FP in `fn_to_numeric_cast_with_truncation`
new_without_default should not warn about unsafe new
fix command to manually test an example
Add license to README
Adding more detail to filter_map lint documentation.
additional people
Add license header to other files
Add license header to Rust files
Relicense clippy
Document relicensing process
Fix util/export.py to include lints from methods
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Oct 13, 2018
@matthiaskrgr
submodules: update clippy from 32b1d1f to 9d33731
ce61be8 
Changes:

Remove now-useless `allow(unknown_lints)`
Stabilize tool lints
Use `impl Iterator` in arg position in clippy_dev
Fix fn_to_numeric_cast_with_truncation suppression
Limit commutative assign op lint to primitive types
Clarify code
Fix rust-lang#2937
Fix cast_possible_wrap and cast_sign_loss warnings
Fix cast_possible_truncation warnings
Fixes rust-lang#2925 cmp_owned false positive
if_let_redundant_pattern_matching: use Span.to() instead of Span.with_hi() to fix crash.
Improve diagnostics in case of lifetime elision (closes rust-lang#3284)
Fix items_after_statements for `const`s
Fix items_after_statements for sub-functions
Fix items_after_statements for `use` statements
Don't suggest cloned() for map Box deref
Fix excessive_precision false positive
Fix FP in `fn_to_numeric_cast_with_truncation`
new_without_default should not warn about unsafe new
fix command to manually test an example
Add license to README
Adding more detail to filter_map lint documentation.
additional people
Add license header to other files
Add license header to Rust files
Relicense clippy
Document relicensing process
Fix util/export.py to include lints from methods
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-resolve Area: Name/path resolution done by `rustc_resolve` specifically
Projects
None yet
Milestone
0.5
Development

No branches or pull requests
2 participants
@catamorphism @bstrie