about summary refs log tree commit diff
path: root/src/test/ui/thinlto
diff options
context:
space:
mode:
authorMazdak Farrokhzad <twingoow@gmail.com>2019-01-15 12:42:04 +0100
committerGitHub <noreply@github.com>2019-01-15 12:42:04 +0100
commitc4b8735872c127f749c0642bc7b8c3a44a2cbb5d (patch)
tree9a7a9ada2a1b471fee3f299c8dbc1936c8c96bf9 /src/test/ui/thinlto
parent33e6df4b62237af312bf6e3f40a97f5bdc94949a (diff)
parent914515f8090be23e7ae0acc29cdccf6360d1a03d (diff)
downloadrust-c4b8735872c127f749c0642bc7b8c3a44a2cbb5d.tar.gz
rust-c4b8735872c127f749c0642bc7b8c3a44a2cbb5d.zip
Rollup merge of #56044 - matthewjasper:function-param-drop-order, r=cramertj
Drop partially bound function parameters in the expected order

Given the function

```rust
fn foo((_x, _): (LogDrop, LogDrop), (_, _y): (LogDrop, LogDrop)) {}
```

Prior to 1.12.0 we dropped both `_x` and `_y` before the rest of their
respective parameters, since then we dropped `_x` and `_y` after. The
original order appears to be the correct order, as the value created
later is dropped first, so we revert to that order and add a test for
it.

While this is technically a breaking change, I can't work out how
anyone could be relying on this without making their code very
brittle. If this is considered to be too likely to break real world code
then I can revert the change and change the test to check for the
current order.
Diffstat (limited to 'src/test/ui/thinlto')
0 files changed, 0 insertions, 0 deletions