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

Add examples for std::ascii #23665

Merged
merged 1 commit into from
Mar 28, 2015
Merged

Conversation

steveklabnik
Copy link
Member

Also tweaked a few things.

Also tweaked a few things.
@rust-highfive
Copy link
Collaborator

r? @brson

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

@@ -246,7 +324,7 @@ pub struct EscapeDefault {
data: [u8; 4],
}

/// Returns a 'default' ASCII and C++11-like literal escape of a `u8`
/// Returns an iterator that produces an escaped version of a `u8`.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'default' is mentioned later too, could it be given some better context/antecedent?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure what it means, actually. But you're right it needs one...

@Manishearth
Copy link
Member

r=me with or without that nit fixed :)

@steveklabnik
Copy link
Member Author

Ehhh let's just leave the nit for now

@steveklabnik
Copy link
Member Author

@bors: r=manishearth rollup

@bors
Copy link
Contributor

bors commented Mar 27, 2015

📌 Commit 528a5e2 has been approved by manishearth

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Mar 27, 2015
@bors bors merged commit 528a5e2 into rust-lang:master Mar 28, 2015
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.

5 participants