Skip to content

Conversation

llogiq and others added 21 commits March 15, 2017 07:24
* Change `utf8` variable names to `non_ascii` to be more clear, since
ASCII and UTF-8 are compatible.

* Fix `EscapeDefault` struct description to follow the typical iterator
method format with a link to the generating function.

* Add more `escape_default` examples to cover every case mentioned in
the function description itself.
This commit stabilizes the `#![windows_subsystem]` attribute which is a
conservative exposure of the `/SUBSYSTEM` linker flag on Widnows platforms. This
is useful for creating applications as well as console programs.

Closes rust-lang#37499
fixed some clippy warnings in libcollections

Some markdown fixes, a few readability improvements, but mostly trivial stuff.
…ystem, r=aturon

rustc: Stabilize the `#![windows_subsystem]` attribute

This commit stabilizes the `#![windows_subsystem]` attribute which is a
conservative exposure of the `/SUBSYSTEM` linker flag on Widnows platforms. This
is useful for creating applications as well as console programs.

Closes rust-lang#37499
… r=alexcrichton

Add a note about overflow for fetch_add/fetch_sub

Fixes rust-lang#40916

r? @steveklabnik
API Docs: ascii

Add/update docs for the `ascii` module per rust-lang#29341.

r? @steveklabnik
…xcrichton

Added links to from_utf8 methods in Utf8Error

Referencing  rust-lang#29375. Linked the `from_utf8` methods for both `String` and `str` in the description. Also linked the `u8` to its documentation
…teveklabnik

Added links to types in from_utf8 description

References rust-lang#29375. Link to types mentioned in the documentation for `from_utf8` (`str`, `&[u8`], etc). Paragraphs were reformatted to keep any one line from being excessively long, but are otherwise unchanged.
Improve documentation for `std::fs::DirBuilder`
@rust-highfive
Copy link
Contributor

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @brson (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see the contribution instructions for more information.

@frewsxcv
Copy link
Contributor Author

frewsxcv commented Apr 2, 2017

@bors r+ p=10

@bors
Copy link
Collaborator

bors commented Apr 2, 2017

📌 Commit d9a9dc4 has been approved by frewsxcv

@bors
Copy link
Collaborator

bors commented Apr 2, 2017

⌛ Testing commit d9a9dc4 with merge 8340dde...

@bors
Copy link
Collaborator

bors commented Apr 2, 2017

💔 Test failed - status-travis

@frewsxcv frewsxcv closed this Apr 2, 2017
@frewsxcv frewsxcv deleted the rollup branch April 2, 2017 14:06
@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rollup A PR which is a rollup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants