Skip to content

NLL: Update box insensitivity test - #55801

Merged
bors merged 3 commits into
rust-lang:masterfrom
pnkfelix:update-box-insensitivity-test-for-nll
Nov 11, 2018
Merged

NLL: Update box insensitivity test#55801
bors merged 3 commits into
rust-lang:masterfrom
pnkfelix:update-box-insensitivity-test-for-nll

Conversation

@pnkfelix

Copy link
Copy Markdown
Contributor

This is just keeping one of our tests honest with respect to NLL, in two ways:

  1. Adds uses of borrows that would otherwise be too short to observe the error that we would have expected to see...
  2. ... I say "would have expected" because all of the errors in this file are part of the reversion of RFC to remove some special treatment of Box by borrow checker rfcs#130 that is attached to NLL (you can see more discussion of this here non-lexical lifetimes (NLL) tracking issue #43234 (comment) )

(The commit prior to this actual passes our test suite, "thanks"
to rust-lang#55695. But since I am aware of that bug, I took advantage of it
in choosing how to order my commit series...)
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 8, 2018
@pnkfelix

Copy link
Copy Markdown
ContributorAuthor

r? @davidtwco

@rust-highfiverust-highfive assigned davidtwco and unassigned aturonNov 8, 2018
@davidtwco

Copy link
Copy Markdown
Member

@bors r+ rollup

@bors

bors commented Nov 9, 2018

Copy link
Copy Markdown
Collaborator

📌 Commit 9b6a568 has been approved by davidtwco

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 9, 2018
emilyalbini added a commit to emilyalbini/rust that referenced this pull request Nov 10, 2018
…est-for-nll, r=davidtwco
NLL: Update box insensitivity test
This is just keeping one of our tests honest with respect to NLL, in two ways:
1. Adds uses of borrows that would otherwise be too short to observe the error that we would have expected to see...
2. ... I say "would have expected" because all of the errors in this file are part of the reversion of rust-lang/rfcs#130 that is attached to NLL (you can see more discussion of this here rust-lang#43234 (comment) )
bors added a commit that referenced this pull request Nov 11, 2018
Rollup of 17 pull requests
Successful merges:
- #55630 (resolve: Filter away macro prelude in modules with `#[no_implicit_prelude]` on 2018 edition)
- #55687 (Take supertraits into account when calculating associated types)
- #55745 (Convert `outlives_components`' return value to a `SmallVec` outparam.)
- #55764 (Fix Rc/Arc allocation layout)
- #55792 (Prevent ICE in const-prop array oob check)
- #55799 (Removed unneeded instance of `// revisions` from a lint test)
- #55800 (Fix ICE in `return_type_impl_trait`)
- #55801 (NLL: Update box insensitivity test)
- #55802 (Don't inline virtual calls (take 2))
- #55816 (Use `SmallVec` to avoid allocations in `from_decimal_string`.)
- #55819 (Typecheck patterns of all match arms first, so we get types for bindings)
- #55822 (ICE with #![feature(nll)] and elided lifetimes)
- #55828 (Add missing `rustc_promotable` attribute to unsigned `min_value` and `max_value`)
- #55839 (Fix docstring spelling mistakes)
- #55844 (Fix documentation typos.)
- #55845 (Set BINARYEN_TRAP_MODE=clamp)
- #55856 (rustdoc: refactor: move all static-file include!s into a single module)
@bors
bors merged commit 9b6a568 into rust-lang:masterNov 11, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@pnkfelix@rust-highfive@davidtwco@bors@aturon