rust-highfive · GitHub

The signature for `std::slice::binary_search` is now parameterized over `Borrow`, as
in `core::SliceExt`

@circuitfox

@circuitfox

@circuitfox

Closed

Closed

@circuitfox

Merged

bors added a commit that referenced this pull request

Aug 20, 2017
Remove Borrow bound from SliceExt::binary_search
#37761 added a Borrow bound to `binary_search` and `binary_search_by_key` in `core::SliceExt`, but did not add it to the methods in `std::slice`. #41590 attempted to add this bound to `std::slice` but was not merged due to breakage. This PR removes the bound in `core::SliceExt`, so that these methods will have the same signature in `core` and `std`.
Fixes #41561

alexcrichton added a commit to alexcrichton/rust that referenced this pull request

Sep 16, 2017
…ig, r=alexcrichton
Remove Borrow bound from SliceExt::binary_search
rust-lang#37761 added a Borrow bound to `binary_search` and `binary_search_by_key` in `core::SliceExt`, but did not add it to the methods in `std::slice`. rust-lang#41590 attempted to add this bound to `std::slice` but was not merged due to breakage. This PR removes the bound in `core::SliceExt`, so that these methods will have the same signature in `core` and `std`.
Fixes rust-lang#41561

bors added a commit that referenced this pull request

Sep 16, 2017
…richton
Remove Borrow bound from SliceExt::binary_search
#37761 added a Borrow bound to `binary_search` and `binary_search_by_key` in `core::SliceExt`, but did not add it to the methods in `std::slice`. #41590 attempted to add this bound to `std::slice` but was not merged due to breakage. This PR removes the bound in `core::SliceExt`, so that these methods will have the same signature in `core` and `std`.
Fixes #41561

Read the original on github.com ↗