libstd: Implement StrBuf, a new string buffer type like Vec, and
#13269
Conversation
|
This looks great, nice work! In general I would like to see lots more documentation on |
|
I wold also be very tempted to put |
|
It might be a good idea to rename the |
|
+1 for rename
|
|
re=r? @alexcrichton |
|
One small comment, otherwise r=me |
port all code over to use it.
|
Rebased in #13440. |
libstd: Implement `StrBuf`, a new string buffer type like `Vec`, and port all code over to use it. Rebased & tests-fixed version of #13269
libstd: Implement `StrBuf`, a new string buffer type like `Vec`, and port all code over to use it. Rebased & tests-fixed version of rust-lang/rust#13269
Properly set the enum variant body type from the repr attribute Follow up for rust-lang/rust-analyzer#12966, fixes some type inference problems
…Alexendoo Rewrite lints page This PR has multiple goals: * Make lints page to work without needing a web server by removing the json file. * Prepare the field to also make the page work with JS (not done in this PR but should be straightforward). * Remove angular dependency. r? `@Alexendoo` changelog: make lint page work without web server
Documentation fixes - In [the page describing the lints](https://rust-lang.github.io/rust-clippy/master/index.html), the View Source links are incorrect. This PR removes the extra segment causing them to fail. - Example before: https://github.com/rust-lang/rust-clippy/blob/master/clippy_lints/clippy_lints/src/absolute_paths.rs#L13 - Example after: https://github.com/rust-lang/rust-clippy/blob/master/clippy_lints/src/absolute_paths.rs#L13 - I think this was introduced in rust-lang#13269. - I've only checked a few of the lints with the new template, but from what I can tell the metadata is generated in the same way for all of them. The `id_location` contains the full path of the lint declaration in the repository, which is why `clippy_lints` was repeated in the URL. - Separately, fixing a typo in the explanation of `unnecessary_get_then_check`. changelog: none
port all code over to use it.
r? @alexcrichton