Skip to content
New issue

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

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

Already on GitHub? # to your account

Simplify CString doc comment #25516

Merged
merged 1 commit into from
May 17, 2015
Merged

Simplify CString doc comment #25516

merged 1 commit into from
May 17, 2015

Conversation

bstrie
Copy link
Contributor

@bstrie bstrie commented May 17, 2015

No need for &b"foo"[..] to make a CString, "foo" will do.

No need for `&b"foo"[..]` to make a CString, `"foo"` will do.
@rust-highfive
Copy link
Contributor

r? @pcwalton

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

@bstrie
Copy link
Contributor Author

bstrie commented May 17, 2015

r? @steveklabnik

@alexcrichton
Copy link
Member

@bors: r+ e1f73b8 rollup

Nice catch!

Manishearth added a commit to Manishearth/rust that referenced this pull request May 17, 2015
No need for `&b"foo"[..]` to make a CString, `"foo"` will do.
bors added a commit that referenced this pull request May 17, 2015
@bors bors merged commit e1f73b8 into rust-lang:master May 17, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants