Skip to content
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.

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

Tiny PR: update a the ByteInput comment #413

Merged
merged 1 commit into from
Nov 8, 2017

Conversation

ethanpailes
Copy link
Contributor

At some ByteInput switched to being backed by actual bytes
rather than a string. This commit just removes the dangling
reference to the old representation.

I just noticed this while reading through code, so I thought I would send the patch your way.

At some ByteInput switched to being backed by actual bytes
rather than a string. This commit just removes the dangling
reference to the old representation.
@BurntSushi
Copy link
Member

Excellent, thank you! @bors r+

@bors
Copy link
Contributor

bors commented Oct 31, 2017

📌 Commit 98ef163 has been approved by BurntSushi

@bors
Copy link
Contributor

bors commented Oct 31, 2017

⌛ Testing commit 98ef163 with merge ff987f6...

bors added a commit that referenced this pull request Oct 31, 2017
Tiny PR: update a the ByteInput comment

At some ByteInput switched to being backed by actual bytes
rather than a string. This commit just removes the dangling
reference to the old representation.

I just noticed this while reading through code, so I thought I would send the patch your way.
@bors
Copy link
Contributor

bors commented Oct 31, 2017

💔 Test failed - status-travis

@ethanpailes
Copy link
Contributor Author

I tried to reproduce that test failure by switching to beta and running both cargo test and ./ci/script.sh, but I coudn't reproduce the failure.

@BurntSushi
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented Nov 8, 2017

⌛ Testing commit 98ef163 with merge cd4ebd3...

bors added a commit that referenced this pull request Nov 8, 2017
Tiny PR: update a the ByteInput comment

At some ByteInput switched to being backed by actual bytes
rather than a string. This commit just removes the dangling
reference to the old representation.

I just noticed this while reading through code, so I thought I would send the patch your way.
@bors
Copy link
Contributor

bors commented Nov 8, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: BurntSushi
Pushing cd4ebd3 to master...

@bors bors merged commit 98ef163 into rust-lang:master Nov 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants