Skip to content
This repository was archived by the owner on May 23, 2024. It is now read-only.

Add test files for several btwn 22354 and 22919 #49

Merged
merged 1 commit into from
Oct 25, 2015
Merged

Add test files for several btwn 22354 and 22919 #49

merged 1 commit into from
Oct 25, 2015

Conversation

rgardner
Copy link
Contributor

The following tests in this range were not added: 22354, 22387,
22465, 22713, 22781, 22872, 22874. Where possible, the tests come
from the shortest possible code samples to reproduce the ICE.

The following tests were not added: 22354, 22387 22465, 22713, 22781,
22872, 22874. Where possible, the tests come from the shortest
possible code samples to reproduce the ICE.
@steveklabnik
Copy link
Member

Thank you so much!

steveklabnik added a commit that referenced this pull request Oct 25, 2015
Add test files for several btwn 22354 and 22919
@steveklabnik steveklabnik merged commit 02fd028 into rust-lang:master Oct 25, 2015
@rgardner rgardner deleted the test-22387-to-22919 branch October 25, 2015 23:21
@wheals
Copy link
Contributor

wheals commented Oct 25, 2015

It looks like the ones that got checked off on the tracking issue were the ones listed here as not added, and vice versa?

@rgardner
Copy link
Contributor Author

Happy to! I definitely want to take a look at some of the trickier ones (e.x. rust-lang/rust#22354) that will take some more time to understand what's going on from the submitted code sample.

@steveklabnik: this PR added tests for 22565, 22566, 22603, 22789, 22894, and 22919. Sorry for the confusion.

@steveklabnik
Copy link
Member

Oh! I mis-read, darn. Thanks @wheals , I'll fix it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants