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

Fix two invalid import cases #195

Closed
wants to merge 1 commit into from
Closed

Fix two invalid import cases #195

wants to merge 1 commit into from

Conversation

espindola
Copy link

Unfortunately this fails for the imports in the use test. I should add parse-only and resolve-only flags to rustc, but for now it looked better to just comment those lines while the import support is finished.

* If an import was unused we would never print any errors for it.
* We would keep the existing environment in scope when descending 'foo.bar'
  and would find 'bar' in the global environment if there was no 'bar' in
  'foo'.
@graydon
Copy link
Contributor

graydon commented Jan 11, 2011

Handled by the other pull req. Thanks.

oli-obk pushed a commit to oli-obk/rust that referenced this pull request Jul 19, 2017
Test the Rc::{into,from}_raw roundtrip
kazcw pushed a commit to kazcw/rust that referenced this pull request Oct 23, 2018
* refactor the x86 module

* document the i686 check

* document strict and intel_sde feature

* document nvptx module
dlrobertson pushed a commit to dlrobertson/rust that referenced this pull request Nov 29, 2018
djtech-dev pushed a commit to djtech-dev/rust that referenced this pull request Dec 9, 2021
…ng_lot-0.11

Update parking_lot requirement from 0.10 to 0.11
matthiaskrgr pushed a commit to matthiaskrgr/rust that referenced this pull request Mar 7, 2023
celinval pushed a commit to celinval/rust-dev that referenced this pull request Jun 4, 2024
…#195)

* Move `CurrentFnCtx` into its own file
* Replace `GotocCtx` forwarding functions with a direct call to the
  `CurrentFnCtx` functions
* Make all `CurrentFnCtx` fields private, with getters and setters.
This pull request was closed.
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