suggest: Put the `use` in suggested code inside the quotes

Change import a trait suggestion from:

   help: candidate #1: use `std::io::Write`

to

   help: candidate #1: `use std::io::Write`

so that the code can be copied directly.

doc: follow the idiom of adding a trailing comma

doc: that explanation was a mess

doc: add missing comma

Auto merge of #30856 - mneumann:thread_local_extern, r=alexcrichton

This will correctly add the thread_local attribute to the external static variable ```errno```:

```rust
extern {
     #[thread_local]
     static errno: c_int;
}
```

Before this commit, the thread_local attribute is ignored. Fixes #30795.

Thanks @alexcrichton for pointing out the solution.

add comment and stop creating directories in runtest

create the required directories while gathering tests rather than during
parallel execution

Auto merge of #31882 - Manishearth:rollup, r=Manishearth

- Successful merges: #31362, #31793, #31800, #31809, #31818, #31827, #31831, #31835, #31837, #31846
- Failed merges:

Rollup merge of #31837 - mitaa:rdoc-inherent-assoc, r=alexcrichton

This effectively only records associated items from either inherent impls or trait definitions in the search-index.

fixes #31808

r? @alexcrichton

Rollup merge of #31835 - mitaa:rdoc-global-src, r=alexcrichton

fixes #26995

r? @alexcrichton

Rollup merge of #31831 - tormol:master, r=alexcrichton

The "A buffer that's too small" example was calling encode_utf**8**().

Rollup merge of #31827 - teoryn:patch-1, r=brson

Rollup merge of #31818 - GuillaumeGomez:error_display, r=brson

Fixes #31788

Add f32 and f64 examples warning

translate backslashes to forward slashes in the expected strings

Document the null-char/null-byte escape in the reference

It appears in the examples, but is not covered by any of the cases
in the prose description.

Rollup merge of #31800 - alexcrichton:armv6-plz, r=brson

Right now the compiler's we're using actually default to armv7/thumb2 I believe,
so this should help push them back to what the arm-unknown-linux-* targets are
for. This at least matches that clang does for the `arm-unknown-linux-gnueabihf`
target which is to map it to an armv6 architecture.

Closes #31787

Rollup merge of #31793 - alexcrichton:add-real-option-gating, r=nikomatsakis

This commit adds support for *truly* unstable options in the compiler, as well
as adding warnings for the start of the deprecation path of
unstable-but-not-really options. Specifically, the following behavior is now in
place for handling unstable options:

* As before, an unconditional error is emitted if an unstable option is passed
  and the `-Z unstable-options` flag is not present. Note that passing another
  `-Z` flag does not require passing `-Z unstable-options` as well.
* New flags added to the compiler will be in the `Unstable` category as opposed
  to the `UnstableButNotReally` category which means they will unconditionally
  emit an error when used on stable.
* All current flags are in a category where they will emit warnings when used
  that the option will soon be a hard error.

Also as before, it is intended that `-Z` is akin to `#![feature]` in a crate
where it is required to unlock unstable functionality. A nightly compiler which
is used without any `-Z` flags should only be exercising stable behavior.

Rollup merge of #31362 - jseyfried:fix_extern_crate_visibility, r=nikomatsakis

This PR changes the visibility of extern crate declarations to match that of items (fixes #26775).
To avoid breakage, the PR makes it a `public_in_private` lint to reexport a private extern crate, and it adds the lint `inaccessible_extern_crate` for uses of an inaccessible extern crate.

The lints can be avoided by making the appropriate `extern crate` declaration public.

mention debug-assertions

Prefer 'match' pattern guard over conditional within body.

Prefer `slice::get` over length check with indexing.

rand: Fix filling buffers 4 GiB or larger with OsRng::fill_bytes on Windows

CryptGenRandom takes a DWORD (u32) for the length so it only supports
writing u32::MAX bytes at a time.

Casting the length from a usize caused truncation meaning the whole
buffer was not always filled.

Move the borrowck run-pass/compile-fail tests into their own directories
as a test.

Add compiletest-ignore-dir to existing subdirectories.

Recurse to find test files in any subdirectory of the base path.  If a
subdirectory contains `compiletest-ignore-dir`, then ignore it.

Generalize from passing around a single path to the test file to passing
around a set of paths called `TestPaths`

This commit is not quite standalone; it basically contains all the
borrowing plumbing bits, the interesting stuff comes in the next commit.

Auto merge of #31873 - Manishearth:rollup, r=Manishearth

- Successful merges: #31677, #31784, #31839, #31842, #31843, #31850, #31863, #31868, #31870
- Failed merges:

Rollup merge of #31870 - ivan:filter-explain, r=steveklabnik

As a Rust newbie, I found the book's explanation for why the `filter` closure gets a reference very confusing, and tried to figure out why `filter` is somehow less consumptive than `map` -- but it isn't; that's controlled by `iter`/`into_iter`.  I flailed around for a while until @habnabit explained it to me, and in retrospect it is quite obvious :-)

Rollup merge of #31868 - fhahn:capitalize-Rust, r=steveklabnik

Rollup merge of #31863 - matklad:clarify-reference, r=steveklabnik

Reference implied that use declarations may appear *only* at the top of blocks and modules, but it is not the case, and the following is valid:

```Rust
fn foo() {
    let x = 92;
    use baz::bar;
}
```

r? @steveklabnik

Rollup merge of #31850 - GuillaumeGomez:vec-doc, r=steveklabnik

r? @steveklabnik
cc @mbrubeck

Rollup merge of #31843 - Wafflespeanut:bool_docs, r=steveklabnik

fixes #29332

r? @steveklabnik

Rollup merge of #31842 - dileepbapat:master, r=alexcrichton

I have made changes and built it after that. Please advise,

https://github.com/rust-lang/rust/issues/31820

Rollup merge of #31839 - mitaa:rdoc-idx-self, r=alexcrichton

fixes #31598

r? @alexcrichton

Rollup merge of #31784 - urschrei:chunks_doc, r=steveklabnik

Closes #31773

r? @steveklabnik

Rollup merge of #31677 - ivan:doc-vtable, r=steveklabnik

I sure hope this fix is right

cc @steveklabnik

Add more explanation on vec type

book: Explain better why the filter closure gets a reference

rustc: Refactor how unstable flags are handled

This commit adds support for *truly* unstable options in the compiler, as well
as adding warnings for the start of the deprecation path of
unstable-but-not-really options. Specifically, the following behavior is now in
place for handling unstable options:

* As before, an unconditional error is emitted if an unstable option is passed
  and the `-Z unstable-options` flag is not present. Note that passing another
  `-Z` flag does not require passing `-Z unstable-options` as well.
* New flags added to the compiler will be in the `Unstable` category as opposed
  to the `UnstableButNotReally` category which means they will unconditionally
  emit an error when used on stable.
* All current flags are in a category where they will emit warnings when used
  that the option will soon be a hard error.

Also as before, it is intended that `-Z` is akin to `#![feature]` in a crate
where it is required to unlock unstable functionality. A nightly compiler which
is used without any `-Z` flags should only be exercising stable behavior.

Capitalize some occurences of Rust in documentation

Fix source-links for files with absolute-paths

`clean_srcpath` tries to make the source-path relative to `src_root`,
but this didn't work since `src_root` itself wasn't absolute.

Allow creation of src-links for device files

std: Cap read/write limits on Windows networking

Similar to #31825 where the read/write limits were capped for files, this
implements similar limits when reading/writing networking types. On Unix this
shouldn't affect anything because the write size is already a `usize`, but on
Windows this will cap the read/write amounts to `i32::max_value`.

cc #31841

reference: clarify use declaration location

Auto merge of #31832 - mmatyas:update_libc_16f1c19, r=alexcrichton

This fixes the `aarch64-linux-android` build (#31595).

Improve unused import detection

Uncapitalize note messages

Auto merge of #31828 - frewsxcv:sepseq-associated-function, r=alexcrichton

None

Use a character that requires two `u16`s in the examples for `char.encode_utf16()`

Auto merge of #31782 - pitdicker:clean_out_windows_c, r=alexcrichton

I am not entirely sure I have got everything right, but if it compiles it is ok probably...
I tested it with msvc x86_64 and gnu.

Somehow a lot of `EXCEPTION-*` constants are dead code when running test, no idea why.
I have put `#![cfg_attr(test, allow(dead_code))]` at the top for this.

Resolve: include normal modules in the ribs

Auto merge of #31778 - aturon:snapshot, r=alexcrichton

r? @alexcrichton

Auto merge of #31855 - alexcrichton:fix-tidy, r=nikomatsakis

The MinGW-based Python implementations would automatically do this, but if we
want to use Python from the official downloads our usage of `/` instead of `\`
can wreak havoc. In a few select locations just use `os.path.normpath` do do the
conversions properly for us.

mk: Tweak tidy script to work on Windows python

The MinGW-based Python implementations would automatically do this, but if we
want to use Python from the official downloads our usage of `/` instead of `\`
can wreak havoc. In a few select locations just use `os.path.normpath` do do the
conversions properly for us.

Warn instead of error when using an inaccessable extern crate

Warn when reexporting a private extern crate

Fix the visibility of extern crate declarations and stop warning on pub extern crate

Show associated types in inherent impls

Don't show associated consts from trait impls

Auto merge of #30614 - arcnmx:cstr-bytes, r=alexcrichton

I'm a bit iffy on the return type, but `Result` would also be a bit weird... The two error cases are `Unterminated` and `InteriorNul(usize)`.

I considered `from_chars(&[c_char])` but this meshes better with `as_bytes_with_nul()` and Rust code in general.

Should there also be a corresponding unsafe `from_bytes_unchecked` variant?

Updated the primitive docs for bool

Auto merge of #31751 - gkoz:os_str_path_cmp, r=aturon

CStr::from_bytes_with_nul tests

#31820 - Utilize `if..let` instead of single `match` branch

Register new snapshots

Link cstr from_bytes to tracking issue

Auto merge of #31825 - ollie27:win_lfs, r=alexcrichton

`ReadFile` and `WriteFile` take a DWORD (u32) for the length argument
which was erroneously cast from a usize causing truncation. This meant
methods like `write_all` and `read_exact` would unexpectedly fail if
given a buffer 4 GiB or larger.

We can instead just ask for `u32::MAX` bytes if the given buffer is too
big.

Don't record self parameter for static methods

Auto merge of #31447 - nagisa:mir-cfgsimplify-bitvec, r=nikomatsakis

BitVector is more space efficient.

[MIR] Change SimplifyCfg pass to use bitvec

BitVector is much more space efficient.

Auto merge of #31822 - petrochenkov:derive, r=alexcrichton

Fixes https://github.com/rust-lang/rust/issues/29548

Rename CStr::from_bytes to from_bytes_with_nul

CStr::from_bytes

Auto merge of #31814 - petevine:master, r=alexcrichton

The `vfp2` option was a leftover from `armv6` compatibility features of the original armhf target.
Gcc defaults to `vfp3`on `armv7` hard-float linux systems so we should make it the default for rustc too.

Correct char.encode_utf16() documentation

The "A buffer that's too small" example was calling encode_utf8().

Use associated functions for libsyntax SepSeq constructors.

Fix warn(unused_mut) in example

Fix reading/writing 4 GiB or larger files on Windows 64-bit

`ReadFile` and `WriteFile` take a DWORD (u32) for the length argument
which was erroneously cast from a usize causing truncation. This meant
methods like `write_all` and `read_exact` would unexpectedly fail if
given a buffer 4 GiB or larger.

We can instead just ask for `u32::MAX` bytes if the given buffer is too
big.

Some refactoring in deriving/debug.rs

Auto merge of #31715 - mitaa:rdoc-index-crate, r=alexcrichton

This allows to search for crates in documentation and simplifies the json serialization of the search-index.

fixes #14077

Auto merge of #31704 - tbu-:pr_vec_into_iter_clone, r=aturon

Auto merge of #30969 - Amanieu:extended_atomic_cmpxchg, r=alexcrichton

This is an implementation of rust-lang/rfcs#1443.

Simplify search-index serialization

Add a test to make sure trans doesn't choke on any of the intrinsics

Fix #[derive] for empty structs with braces

Auto merge of #31819 - miqid:doc, r=steveklabnik

Hello.

I came across some minor spelling mistakes while going through the documentation for the `Iterator` trait. Also includes a suggested styling fix.

r? @steveklabnik

Update libc to 16f1c19

Correct Iterator trait documentation

Fixes several minor spelling errors and includes a suggested style fix.

Ignore test case on Windows.

Windows is not #[cfg(target_thread_local)] and as such should link
to the external symbol. But it fails with:

    thread '<main>' panicked at 'assertion failed: `(left == right)` (left: `272246271`, right: `3`)', C:/bot/slave/auto-win-msvc-64-opt/build/src/test/run-pass/thread-local-extern-static.rs:24

Eradicate last vestiges of armv6

Auto merge of #31813 - nbaksalyar:solaris-fix, r=sanxiyn

A quick fix for several issues that break a Solaris/Illumos build.
Also, adds a CPU target specification (as seen in a patch for OpenBSD #31727).

Auto merge of #31811 - alexcrichton:clean-deps, r=sanxiyn

The standard library doesn't depend on rustc_bitflags, so move it to explicit
dependencies on all other crates. Additionally, the arena/fmt_macros deps could
be dropped from libsyntax.

Auto merge of #31805 - cuviper:android-lfs, r=alexcrichton

Android should use 64-bit LFS symbols for `lseek` and `ftruncate`, lest
those offset parameters suffer a lossy cast down to a 32-bit `off_t`.

Unlike GNU/Linux, Android's `stat`, `dirent`, and related functions are
always 64-bit LFS compatible, and `open` already implies `O_LARGEFILE`,
so all those don't need to follow Linux.  It might be nice to unify them
anyway, but those other LFS symbols aren't present in API 18 bionic.

r? @alexcrichton

Auto merge of #31806 - gokhanettin:to_lowercase-doc, r=japaric

`to_lowercase` example mistakenly calls `to_uppercase` function. This fixes the example.

Auto merge of #31810 - gandro:netbsd-fix-stat, r=alexcrichton

Some struct members have a slighty different name on NetBSD. This has been fixed in the libc crate, but not in libstd, breaking the NetBSD build. Related C struct definition: http://nxr.netbsd.org/xref/src/sys/sys/stat.h?r=1.68#59

This also removes the broken `st_spare()` from MetadataExt, since it is private field reserved for future use.

@dhuseby In case this conflicts with any of your pending patches, feel free to intervene - I'm happy to withdraw this PR.

r? @alexcrichton

Fix broken Solaris build

RaiseException is used by everything except x86 gnu

Auto merge of #31792 - Zoxc:sysroot, r=alexcrichton

With these changes you can build a freestanding sysroot without using floating points using a Cargo.toml and copying the `deps` folder cargo outputs.
```
[package]
name = "sysroot"
version = "0.1.0"

[lib]
path = "lib.rs"
crate-type = ["rlib"]

[dependencies.core]
path = "../vendor/rust/src/src/libcore"
features = ["disable_float"]

[dependencies]
collections = { path = "../vendor/rust/src/src/libcollections" }
```

rustbuild: Sync some Cargo.toml/lib.rs dependencies

The standard library doesn't depend on rustc_bitflags, so move it to explicit
dependencies on all other crates. Additionally, the arena/fmt_macros deps could
be dropped from libsyntax.