]> git.lizzy.rs Git - rust.git/commit
Rollup merge of #42428 - scottmcm:str-get-overflow, r=sfackler
authorCorey Farwell <coreyf@rwell.org>
Tue, 13 Jun 2017 21:14:59 +0000 (17:14 -0400)
committerGitHub <noreply@github.com>
Tue, 13 Jun 2017 21:14:59 +0000 (17:14 -0400)
commit78d5d3723567e57e82ba65d73d2bc8bbe7f5c017
tree4143f38d1c339001b14a967c79c1aea3b3f90178
parent02179bd2515fcdc5cdfeecd84b04097f0c80daf2
parent808a08a363591abf754fafd93ec3f44c686486ec
Rollup merge of #42428 - scottmcm:str-get-overflow, r=sfackler

Add overflow checking for `str::get` with inclusive ranges

Fixes https://github.com/rust-lang/rust/issues/42401

Two commits here:

1. The first makes `str::index` just call `SliceIndex<str>::index`.  It's intended to have no behavior change, except where the two methods were inconsistent.
2. The second actually adds the overflow checking to `get(_mut)` (and tests for it)
src/libcollections/tests/lib.rs