diff options
| author | Mazdak Farrokhzad <twingoow@gmail.com> | 2019-01-15 12:42:04 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-01-15 12:42:04 +0100 |
| commit | c4b8735872c127f749c0642bc7b8c3a44a2cbb5d (patch) | |
| tree | 9a7a9ada2a1b471fee3f299c8dbc1936c8c96bf9 /src/test/ui/thinlto | |
| parent | 33e6df4b62237af312bf6e3f40a97f5bdc94949a (diff) | |
| parent | 914515f8090be23e7ae0acc29cdccf6360d1a03d (diff) | |
| download | rust-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
