diff options
| author | bors <bors@rust-lang.org> | 2019-04-17 19:18:51 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2019-04-17 19:18:51 +0000 |
| commit | 3c3d3c1777041200bb7ed7a65b6562d62899778c (patch) | |
| tree | 1475fc81e1b201e1944a5cc843c89470e8587396 /src/libstd/sys/unix/stack_overflow.rs | |
| parent | 8260e9676089363648dfbce072fca562093b4289 (diff) | |
| parent | cc3abc4d2786be6d8366eb904b8ea4ac4a3e9f7e (diff) | |
| download | rust-3c3d3c1777041200bb7ed7a65b6562d62899778c.tar.gz rust-3c3d3c1777041200bb7ed7a65b6562d62899778c.zip | |
Auto merge of #59527 - matklad:sized-index, r=Centril
Add test checking that Index<T: ?Sized> works I've noticed that we have an `Idx: ?Sized` bound on the **index** in the `Index`, which seems strange given that we accept index by value. My guess is that it was meant to be removed in https://github.com/rust-lang/rust/pull/23601, but was overlooked. If I remove this bound, `./x.py src/libstd/ src/libcore/` passes, which means at least that this is not covered by test. I think there's three things we can do here: * run crater with the bound removed to check if there are any regressions, and merge this, to be consistent with other operator traits * run crater, get regressions, write a test for this with a note that "hey, we tried to fix it, its unfixable" * decide, in the light of by-value DSTs, that this is a feature rather than a bug, and add a test cc @rust-lang/libs EDIT: the forth alternative is that there exist a genuine reason why this is the case, but I failed to see it :D
Diffstat (limited to 'src/libstd/sys/unix/stack_overflow.rs')
0 files changed, 0 insertions, 0 deletions
