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 use of "either" in rt.rs and followingly mod.rs #9294

Closed
wants to merge 1 commit into from
Closed

remove use of "either" in rt.rs and followingly mod.rs #9294

wants to merge 1 commit into from

Conversation

wakandan
Copy link

regarding #9157 (#9157) to remove "either", this is my first commit for rt.rs and mod.rs

Great help from https://github.com/huonw

@alexcrichton
Copy link
Member

Sadly I think that you picked one of the tricky cases to tackle first! There are actually no written-down constructors of these variants, but rather they're generated by the format_args! macro. If you use a compiler built with this code, it should fail to compile:

fn main() {
  println!("{0, plural, =1{#} other{no match}}", 1);
}

In addition to updating these modules, I think you'll also need to remove the usage of Left and Right from the syntax expander in src/libsyntax/ext/format.rs.

If you need any help though, feel free to post here or ping me on IRC (I'm acrichto)

@wakandan
Copy link
Author

It's quite a surprise to me, thanks for letting me know this. How could I detect such a case? After modifying these module I was using "make check-stage1-std" to run the tests. Should I re-compile the whole project?

@alexcrichton
Copy link
Member

If you run make check-stage1-rpass TESTNAME=ifmt it should fail unless you've updated it correctly.

@pnkfelix
Copy link
Member

I'm closing this pull request to get it off the bors queue. (Feel free to open another PR if you get this working.)

@pnkfelix pnkfelix closed this Sep 21, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Aug 11, 2022
…windows, r=xFrednet

Enable test for def_id_nocore for windows

Verified that it actully works on windows
changelog: none
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