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

Remove unnecessary 'use' forms #6833

Merged
merged 1 commit into from
May 31, 2013
Merged

Remove unnecessary 'use' forms #6833

merged 1 commit into from
May 31, 2013

Conversation

fdr
Copy link
Contributor

@fdr fdr commented May 30, 2013

Fix a laundry list of warnings involving unused imports that glutted
up compilation output. There are more, but there seems to be some
false positives (where 'remedy' appears to break the build), but this
particular set of fixes seems safe.

Fix a laundry list of warnings involving unused imports that glutted
up compilation output.  There are more, but there seems to be some
false positives (where 'remedy' appears to break the build), but this
particular set of fixes seems safe.
@fdr
Copy link
Contributor Author

fdr commented May 30, 2013

Well, that apparently wasn't so safe after all. I wonder how I didn't catch that one. Made a speculative amendment (already seen) and building on my own machine to verify.

Are there some problems with dependency tracking on incremental builds as-is?

@fdr
Copy link
Contributor Author

fdr commented May 30, 2013

My platform claims all the expected tests pass.

bors added a commit that referenced this pull request May 31, 2013
Fix a laundry list of warnings involving unused imports that glutted
up compilation output.  There are more, but there seems to be some
false positives (where 'remedy' appears to break the build), but this
particular set of fixes seems safe.
@bors bors closed this May 31, 2013
@bors bors merged commit aef1e10 into rust-lang:incoming May 31, 2013
@fdr fdr deleted the fix-warnings branch May 31, 2013 17:56
flip1995 pushed a commit to flip1995/rust that referenced this pull request Mar 11, 2021
Fix false-positive in `use_self`

Fixes  rust-lang#6818.

changelog: Fix false positives for use_self in macros.
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