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

syntax: Make ViewItemUse no longer contain multiple view paths. #13777

Merged
merged 2 commits into from
Apr 27, 2014

Conversation

lifthrasiir
Copy link
Contributor

It reflected the obsolete syntax use a, b, c; and did not make past the parser (though it was a non-fatal error so we can continue). This legacy affected many portions of rustc and rustdoc as well, so this PR cleans them up altogether.

As a side effect of cleanup, we now have SCHEMA_VERSION in rustdoc::clean (instead of the crate root), so it has a better chance to be updated when rustdoc::clean gets updated.

it reflected the obsolete syntax `use a, b, c;` and did not make
past the parser (though it was a non-fatal error so we can continue).
this legacy affected many portions of rustc and rustdoc as well,
so this commit cleans them up altogether.
it should be changed when the module gets updated, so it helps to
have the `SCHEMA_VERSION` with the definitions themselves.
@lifthrasiir
Copy link
Contributor Author

Caveat: While I believe this PR does not affect other crates (and the most important test cases do pass), make check somehow fails on the unexpected tests so they need to be reexamined. In particular,

  • On my Windows machine (with 32-bit MinGW) make check always failed even after reconfigure and make clean, due to the linkage problem from the differing function hash. I'm not very sure about the underlying reason.
  • On x86_64-unknown-linux-gnu host/target, debug-info/function-prologue-stepping-no-split-stack and debug-info/recursive-enum mysteriously fails. (And I should also note that the lack of check-stage2-cfail-full make target drove me crazy, aargh.)

@lifthrasiir lifthrasiir changed the title syntax: ViewItemUse no longer contains multiple view paths. syntax: Make ViewItemUse no longer contain multiple view paths. Apr 26, 2014
@alexcrichton
Copy link
Member

On my Windows machine (with 32-bit MinGW) make check always failed even after reconfigure and make clean, due to the linkage problem from the differing function hash

This happened to me awhile back because I had an otherwise installed version of rust in my PATH, and the linkage was getting mismatched. I don't think this is your fault.

On x86_64-unknown-linux-gnu host/target, debug-info/function-prologue-stepping-no-split-stack and debug-info/recursive-enum mysteriously fails

I had no idea any debuginfo tests passed on windows!

bors added a commit that referenced this pull request Apr 26, 2014
…richton

It reflected the obsolete syntax `use a, b, c;` and did not make past the parser (though it was a non-fatal error so we can continue). This legacy affected many portions of rustc and rustdoc as well, so this PR cleans them up altogether.

As a side effect of cleanup, we now have `SCHEMA_VERSION` in `rustdoc::clean` (instead of the crate root), so it has a better chance to be updated when `rustdoc::clean` gets updated.
bors added a commit that referenced this pull request Apr 26, 2014
…richton

It reflected the obsolete syntax `use a, b, c;` and did not make past the parser (though it was a non-fatal error so we can continue). This legacy affected many portions of rustc and rustdoc as well, so this PR cleans them up altogether.

As a side effect of cleanup, we now have `SCHEMA_VERSION` in `rustdoc::clean` (instead of the crate root), so it has a better chance to be updated when `rustdoc::clean` gets updated.
bors added a commit that referenced this pull request Apr 27, 2014
…richton

It reflected the obsolete syntax `use a, b, c;` and did not make past the parser (though it was a non-fatal error so we can continue). This legacy affected many portions of rustc and rustdoc as well, so this PR cleans them up altogether.

As a side effect of cleanup, we now have `SCHEMA_VERSION` in `rustdoc::clean` (instead of the crate root), so it has a better chance to be updated when `rustdoc::clean` gets updated.
@bors bors closed this Apr 27, 2014
@bors bors merged commit bec77eb into rust-lang:master Apr 27, 2014
arcnmx pushed a commit to arcnmx/rust that referenced this pull request Dec 17, 2022
fix: add a check for `if` token in patterns parser

Closes rust-lang#13776
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