Minor cleanup in tests for str #29279

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

bors merged 2 commits into rust-lang:master from Ryman:strtests

Oct 25, 2015

Contributor

Ryman commented Oct 24, 2015

No description provided.

Ryman added 2 commits

October 24, 2015 19:53


          Remove unnecessary String allocations from str tests

49c7878


          Add assertions to test_total_ord for str

83b308e

rust-highfive assigned nikomatsakis

Contributor

rust-highfive commented Oct 24, 2015

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

Member

alexcrichton commented Oct 24, 2015

@bors: r+ 83b308e

Thanks!

bors added a commit that referenced this pull request


          Auto merge of #29279 - Ryman:strtests, r=alexcrichton

5acda1d

Collaborator

bors commented Oct 25, 2015

⌛ Testing commit 83b308e with merge 5acda1d...

Collaborator

bors commented Oct 25, 2015

☀️ Test successful - auto-linux-32-nopt-t, auto-linux-32-opt, auto-linux-64-debug-opt, auto-linux-64-nopt-t, auto-linux-64-opt, auto-linux-64-x-android-t, auto-linux-cross-opt, auto-linux-musl-64-opt, auto-mac-32-opt, auto-mac-64-nopt-t, auto-mac-64-opt, auto-win-gnu-32-nopt-t, auto-win-gnu-32-opt, auto-win-gnu-64-nopt-t, auto-win-gnu-64-opt, auto-win-msvc-32-opt, auto-win-msvc-64-opt

bors merged commit 83b308e into rust-lang:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet