internal: move to Arc::from_iter

[davidbarsky]

Builds atop of #15985, will rebase.

[davidbarsky]

...and rebased!

[lnicola]

@Veykril want to postpone this until after #15959?

[Veykril]

Would be nice if I didn't have to rebase over this

[lnicola]

It's not as bad as it looks though, mainly Arc::from(x.collect::<Vec<_>>()) to Arc::from_iter(x). But this isn't urgent at all, so we can wait.

[davidbarsky]

Yeah, the change looks substantially noisier than it actually is, but there's no rush on landing this.

[Veykril]

its more it will conflict with my stuff and it touches things that I think i touched over multiple commits. Rebasing isnt really gonna be difficult but I'd also rather not have to rebase my stuff over this if I can avoid it

[davidbarsky]

its more it will conflict with my stuff and it touches things that I think i touched over multiple commits. Rebasing isnt really gonna be difficult but I'd also rather not have to rebase my stuff over this if I can avoid it

yeah, fair. i'll wait to land this until your tokenmap work landed.

[bors]

☔ The latest upstream changes (presumably #16017) made this pull request unmergeable. Please resolve the merge conflicts.

[davidbarsky]

alright, this PR is no longer blocked.

[lnicola]

@bors r+

[bors]

📌 Commit c17dcc8 has been approved by lnicola

It is now in the queue for this repository.

[bors]

⌛ Testing commit c17dcc8 with merge 311a5e9...

[bors]

☀️ Test successful - checks-actions
Approved by: lnicola
Pushing 311a5e9 to master...