Skip to content

Removing iter::TimesIx - #3881

Merged
graydon merged 2 commits into
rust-lang:incomingfrom
vertexclique:incoming
Oct 30, 2012
Merged

Removing iter::TimesIx#3881
graydon merged 2 commits into
rust-lang:incomingfrom
vertexclique:incoming

Conversation

@vertexclique

Copy link
Copy Markdown
Contributor

from uint and int templating

@vertexclique

Copy link
Copy Markdown
ContributorAuthor

Refer to #3130

@brson

Copy link
Copy Markdown
Contributor

We discussed this on IRC, but I'll record it here: this patch removes two impls of TimesIx but not the trait itself. The trait, defined in core::iter, should be removed too.

@vertexclique

Copy link
Copy Markdown
ContributorAuthor

done

@graydon

Copy link
Copy Markdown
Contributor

lgtm, thanks.

graydon added a commit that referenced this pull request Oct 30, 2012
@graydon
graydon merged commit 9bcafa2 into rust-lang:incomingOct 30, 2012
flip1995 pushed a commit to flip1995/rust that referenced this pull request Feb 25, 2021
Rework use_self impl based on ty::Ty comparison rust-lang#3410 | Take 2
This builds on top of rust-lang#5531
I already reviewed and approved the commits by `@montrivo.` So only the review of my commits should be necessary.
I would also appreciate your review `@montrivo,` since you are familiar with the challenges here.
Fixesrust-lang#3410 and Fixesrust-lang#4143 (same problem)
Fixesrust-lang#2843Fixesrust-lang#3859Fixesrust-lang#4734 and fixesrust-lang#6221Fixesrust-lang#4305Fixesrust-lang#5078 (even at expression level now 🎉)
Fixesrust-lang#3881 and Fixesrust-lang#4887 (same problem)
Fixesrust-lang#3909
Not yet: rust-lang#4140 (test added)
All the credit for the fixes goes to `@montrivo.` I only refactored and copy and pasted his code.
changelog: rewrite [`use_self`] lint and fix multiple (8) FPs. One to go.
RalfJung pushed a commit to RalfJung/rust that referenced this pull request Sep 17, 2024
./miri run: directly run binary instead of using 'cargo run'
This avoids re-building miri without dev dependencies, so it makes `./miri run` a lot faster if `./miri test` was already executed before.
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
3881: Add functional update test r=matklad a=matklad
bors r+
🤖
Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>
Sign up for freeto 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

@vertexclique@brson@graydon