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 some obsolete ignored tests #12476

Closed
wants to merge 1 commit into from

Conversation

fhahn
Copy link
Contributor

@fhahn fhahn commented Feb 22, 2014

This PR removes 4 obsolete ignored test cases

* compile-fail/vec-add.rs is obsolete, there are no mutable
  vectors any more, rust-lang#2711 is closed
* compile-fail/issue-1451.rs is obsolete, there are no more
  structural records, rust-lang#1451 is closed
* compile-fail/issue-2074.rs is obsolete, an up to date test
  is in run-pass/nested-enum-same-names.rs, rust-lang#2074 is closed
* compile-fail/omitted-arg-wrong-types.rs is obsolete, rust-lang#2093
  is closed
@alexcrichton
Copy link
Member

This may conflict with #12457 a bit, but I'll let bors figure it out.

@fhahn
Copy link
Contributor Author

fhahn commented Feb 22, 2014

It seems like your cleanup was way more extensive ;)

@bors bors closed this in 4243cad Feb 25, 2014
@fhahn fhahn deleted the ignored-tests-update2 branch February 25, 2014 10:04
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 25, 2022
Restart server automatically on settings changes

Closes rust-lang#12476

I think this works quite well, but if you think it would be better to put it behind a setting I can do that.
flip1995 pushed a commit to flip1995/rust that referenced this pull request Sep 5, 2024
…check, r=y21

Extend `implicit_saturating_sub` lint

Fixes rust-lang#10070.

It can serve as base if we want to add equivalent checks for other arithmetic operations.

Also one important note: when writing this lint, I realized that I could check for wrong conditions performed beforehand on subtraction and added another part in the lint. Considering they both rely on the same checks, I kept both in the same place. Not sure if it makes sense though...

changelog: Extend `implicit_saturating_sub` lint
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