libstd: add example for PathBuf::push

Auto merge of #29684 - stepancheg:size-hint, r=Gankro

* `size_hint()` cannot be relied upon to perform memory unsafe operations

Same applies to `len()` function of `ExactSizeIterator` trait.

Auto merge of #29686 - jrburke:docs-error-handling-search-case, r=BurntSushi

In src/doc/trpl/error-handling.md, in this section:

It mentions three steps, to "convert this to proper error handling", the last one being:

3. Handle the error in main.

However, it is not shown. This pull request adds a code example showing how `main`'s call to `search` should use case analysis. I am still very much new to learning Rust, so this may not be idiomatic. Happy to make changes with guidance.

Auto merge of #29690 - tbu-:pr_doc_env_home_dir, r=Manishearth

Explain that size_hint cannot be trusted

Same applies to `len()` function of `ExactSizeIterator` trait.

doc: error-handling.md: main case analysis for search

Auto merge of #29692 - jwworth:pull-request/fix-typo, r=apasel422

Fixes the double word 'be'.

Fix double word 'be'

Spell out the fallback of `std::env::home_dir` on POSIX

Auto merge of #29683 - apasel422:typos, r=steveklabnik

r? @steveklabnik

Auto merge of #29681 - vadimcn:fix-dist, r=alexcrichton

Resolves  #29672.   This happened because rust runtime startup objects, rsbegin.o and rsend.o, were not included in the target libraries package for -windows-gnu.

r? @alexcrichton

Remove duplicate words from docs

Make sure rsbegin.o and rsend.o get packaged with target lib artifacts.
Also, unified libc startup objects finding logic with that of the `-musl` target, since conceptually they were doing the same thing.

Auto merge of #29674 - inrustwetrust:flag_alignment, r=alexcrichton

The `enable-nonzeroing-move-hints` flag name was too long and caused misalignment of the help text.
Now calculating the needed padding dynamically from the available flags instead.

Auto merge of #29678 - jwworth:pull-request/fix-typo, r=apasel422

Fixes the typo 'is is'.

Fix typo

Auto merge of #29671 - ollie27:patch-1, r=bluss

Fixed text alignment for output of "rustc -Z help"

Update list of array implementations in src/libstd/primitive_docs.rs

Auto merge of #29662 - rust-lang:brson-patch-2, r=steveklabnik

r? @steveklabnik

Direct security bugs to the security policy.

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

- Successful merges: #29617, #29622, #29656, #29659, #29660
- Failed merges:

Rollup merge of #29660 - steveklabnik:gh28461, r=brson

Fixes #28461

Rollup merge of #29659 - angelsl:crt-rebase, r=alexcrichton

cc: @vadimcn @vhbit

r? @alexcrichton

Rollup merge of #29656 - arcnmx:static-extern, r=alexcrichton

This is currently done for functions but not public static symbols.

Rollup merge of #29622 - steveklabnik:gh29621, r=bstrie

Now that thread::sleep is a real thing, let's use it

Fixes #29621

r? @bstrie

Rollup merge of #29617 - steveklabnik:gh29591, r=alexcrichton

Fixes #29591

Auto merge of #29551 - arcnmx:target-family, r=alexcrichton

Allow the changing of `target_family` through flexible configuration. The whole computing world isn't just a binary of *nix and Windows! Makes porting `libstd` and co to new platforms a lot less painful.

Auto merge of #29462 - alexcrichton:refactor-process-ret, r=aturon

* Store the native representation directly in the `ExitStatus` structure instead
  of a "parsed version" (mostly for Unix).
* On Windows, be more robust against processes exiting with the status of 259.
  Unfortunately this exit code corresponds to `STILL_ACTIVE`, causing libstd to
  think the process was still alive, causing an infinite loop. Instead the loop
  is removed altogether and `WaitForSingleObject` is used to wait for the
  process to exit.

std: Refactor process exit code handling slightly

* Store the native representation directly in the `ExitStatus` structure instead
  of a "parsed version" (mostly for Unix).
* On Windows, be more robust against processes exiting with the status of 259.
  Unfortunately this exit code corresponds to `STILL_ACTIVE`, causing libstd to
  think the process was still alive, causing an infinite loop. Instead the loop
  is removed altogether and `WaitForSingleObject` is used to wait for the
  process to exit.

Auto merge of #29582 - oli-obk:token_tree, r=sfackler

sleep_ms -> sleep

Now that thread::sleep is a real thing, let's use it

Fixes #29621

Add multi-line string literals to TRPL

Fixes #29591

Make a note about "gigabyte"

Fixes #28461

Auto merge of #29657 - defuz:patch-2, r=steveklabnik

r? @steveklabnik

Update compiler-rt

Auto merge of #29643 - petrochenkov:stability5, r=alexcrichton

Also remove `stable` stability annotations from inherent impls

(There will be a warning for useless stability annotations soon.)

r? @Gankro

remove `Tt` prefix from TokenType variants

[breaking change]

Auto merge of #29620 - petrochenkov:reachable2, r=alexcrichton

Handle them in `middle::reachable` instead (no optimizations so far, just drop all trait impl items into the reachable set, as before). Addresses the concerns from https://github.com/rust-lang/rust/pull/29291#discussion_r43672413
\+ In `middle::reachable` don't treat impls of `Drop` specially, they are subsumed by the general impl treatment.
\+ Add some tests checking reachability of trait methods written in UFCS form
\+ Minor refactoring in the second commit

r? @alexcrichton

Opps, fix incorrect call

Rename me in AUTHORS.txt

Removing the hack from error handling chapter, which is does not make sense anymore.

Auto merge of #29615 - steveklabnik:lol_strings, r=alexcrichton

&format!("...") is the same as "" if we're not doing any interpolation,
and doesn't allocate an intermediate String.

Preserve public static items across LTO

Auto merge of #29604 - bstrie:de_ms, r=alexcrichton

Auto merge of #29600 - Ryman:rustdoc_better_testnames, r=alexcrichton

Old doctest names
```bash
test sync::atomic::load_0 ... ok
test sync::atomic::load_0 ... ok
test sync::atomic::load_0 ... ok
test sync::atomic::load_0 ... ok
```

New doctest names
```bash
test sync::atomic::AtomicBool::load_0 ... ok
test sync::atomic::AtomicIsize::load_0 ... ok
test sync::atomic::AtomicPtr<T>::load_0 ... ok
test sync::atomic::AtomicUsize::load_0 ... ok
```

Auto merge of #29588 - nikomatsakis:mir-switch, r=aatch

Introduce a `SwitchInt` and restructure pattern matching to collect integers and characters into one master switch. This is aimed at #29227, but is not a complete fix. Whereas before we generated an if-else-if chain and, at least on my machine, just failed to compile, we now spend ~9sec compiling `rustc_abuse`. AFAICT this is basically just due to a need for more micro-optimization of the matching process: perf shows a fair amount of time just spent iterating over the candidate list. Still, it seemed worth opening a PR with this step alone, since it's a big step forward.

Auto merge of #29491 - alexcrichton:avoid-stdio-tls, r=brson

Currently if a print happens while a thread is being torn down it may cause a
panic if the LOCAL_STDOUT TLS slot has been destroyed by that point. This adds a
guard to check and prints to the process stdout if that's the case (as we do for
if the slot is already borrowed).

Closes #29488

Auto merge of #29653 - alexcrichton:fix-distcheck-again-zomg, r=brson

The recent change of libdir on windows was accidentally not propagated to
`make dist` and related commands. This commit touches that up!

Closes #29640

Auto merge of #29305 - alexcrichton:bad-getenv, r=brson

As discovered in #29298, `env::set_var("", "")` will panic, but it turns out
that it *also* deadlocks on Unix systems. This happens because if a panic
happens while holding the environment lock, we then go try to read
RUST_BACKTRACE, grabbing the environment lock, causing a deadlock.

Specifically, the changes made here are:

* The environment lock is pushed into `std::sys` instead of `std::env`. This
  also only puts it in the Unix implementation, not Windows where the functions
  are already threadsafe.
* The `std::sys` implementation now returns `io::Result` so panics are
  explicitly at the `std::env` level.

mk: Account for libdir change on windows

The recent change of libdir on windows was accidentally not propagated to
`make dist` and related commands. This commit touches that up!

Closes #29640

librustdoc: improve testnames for doctests

Auto merge of #29648 - steveklabnik:rollup, r=steveklabnik

- Successful merges: #29609, #29611, #29613, #29614, #29619, #29624, #29630
- Failed merges:

Rollup merge of #29630 - apasel422:marker, r=steveklabnik

r? @steveklabnik

Rollup merge of #29624 - rilut:patch-1, r=steveklabnik

Thanks.
r? @steveklabnik

Rollup merge of #29619 - steveklabnik:gh28824, r=alexcrichton

Fixes #28824

Rollup merge of #29614 - bluss:vec-drop-comment, r=Gankro

vec: Remove old comment in Vec::drop

This comment was leftover from an earlier revision of a PR, something
that never was merged. There is no ZST special casing in Vec::drop.

Rollup merge of #29613 - steveklabnik:29506, r=alexcrichton

in favor of just expect()

Fixes #29506

Rollup merge of #29611 - steveklabnik:gh25918, r=alexcrichton

Originally, this was my 30 minute introduction, and we eventually made
it the opener to the book. But as #25918 has shown, the example I use
here has some issues. The good news is that Rust makes heap allocation
syntatically expensive, but the bad news is that that means showing
equivalent programs from Rust and other languages is difficult. After
thinking about it, I'm not sure this section is pulling its weight, and
since it has problems, I'd rather just pull it than try to re-write it
right now. I think the book is fine without it.

FIxes #25918

Rollup merge of #29609 - ivan:rustdoc-ctrl-s, r=steveklabnik

Rustdoc pages with a search box inadvertently override `ctrl-s` in addition to the intended `s` and `S` keys.  You can test this in at least Firefox and Chrome (tested: Windows): press `ctrl-s` on http://doc.rust-lang.org/std/.  The search box is focused when instead the browser's save feature should be activated.  This PR fixes `ctrl-s` and possibly other browser shortcuts.

Remove stability annotations from trait impl items

Remove `stable` stability annotations from inherent impls

Auto merge of #29583 - dotdash:mir_small_agg, r=nikomatsakis

Fix handling of small aggregate function arguments and assignments of temporaries to lvalues.

Tidy `core::marker` doc summaries

Auto merge of #29565 - sfackler:issue-29540, r=huonw

Closes #29540

r? @huonw

doc(lib.rs): fix #L79 with inline link syntax

Convert TODO into FIXME

Auto merge of #29593 - ben0x539:reflect-doc-comment, r=steveklabnik

Rustdoc takes the first paragraph as a summary, so having a huge
paragraph that ends with introducing an example looked somewhat wrong on
the module page.

Beef up macro designator docs

Fixes #28824

`EmbargoVisitor::reexports` isn't needed

Fix handling of fat pointer function arguments

The store for the "extra" data went to the wrong destination.

rustc_privacy: Do not export items needed solely for the reachability analysis

Process them in middle::reachable instead
Add tests for reachability of trait methods written in UFCS form

remove excess string allocation

&format!("...") is the same as "" if we're not doing any interpolation,
and doesn't allocate an intermediate String.

vec: Remove old comment in Vec::drop

This comment was leftover from an earlier revision of a PR, something
that never was merged. There is no ZST special casing in Vec::drop.

Remove .ok().expect()

in favor of just expect()

Fixes #29506

Remove short intro from README

Originally, this was my 30 minute introduction, and we eventually made
it the opener to the book. But as #25918 has shown, the example I use
here has some issues. The good news is that Rust makes heap allocation
syntatically expensive, but the bad news is that that means showing
equivalent programs from Rust and other languages is difficult. After
thinking about it, I'm not sure this section is pulling its weight, and
since it has problems, I'd rather just pull it than try to re-write it
right now. I think the book is fine without it.

FIxes #25918

Auto merge of #29610 - steveklabnik:rollup, r=steveklabnik

- Successful merges: #29416, #29537, #29538, #29539, #29567, #29568, #29571, #29579
- Failed merges:

docs for Reflect: blank line after first sentence

Rustdoc takes the first paragraph as a summary, so having a huge
paragraph that ends with introducing an example looked somewhat wrong on
the module page.

Rollup merge of #29579 - steveklabnik:gh27322, r=apasel422

Fixes #27322

Rollup merge of #29571 - steveklabnik:gh29322, r=apasel422

Fixes #29322

Rollup merge of #29568 - steveklabnik:gh24591, r=apasel422

Fixes #24591

Rollup merge of #29567 - Toby-S:patch-1, r=steveklabnik

Reword "Writing the logic" paragraph to prevent `unwrap` being confused for a macro (and other small changes to improve the flow of the paragraph).

cc @steveklabnik

Rollup merge of #29539 - shepmaster:empty-iterator-docs, r=Gankro

Rollup merge of #29538 - steveklabnik:ch1_edits, r=brson

I did some preliminary editing work with No Starch on the first chapter of the book, and here's some of the results. We're going to want to return to this later when @brson etc's new rustup work is done, so this is mostly just a first pass.

But, we agreed that having separate chapters for each of this bit of intro is a bit excessive. So let's move all of this intro stuff into one chapter.

I'd appreciate a careful review of this, as there was also some confusion about some things, which resulted in me taking one huge markdown file apart and splitting it back up, as well as some editor issues, so I _think_ this looks good, but double checking things matters!

/cc @aturon

Rollup merge of #29537 - bltavares:issue-24954, r=steveklabnik

Closes #24954

Rollup merge of #29416 - brson:relnotes, r=alexcrichton

[Rendered](https://github.com/brson/rust/blob/relnotes/RELEASES.md)

More highlights would be good. I intend to go through and hyperlink all the APIs still.

librustdoc: don't override ctrl-s and other browser shortcuts

Auto merge of #28887 - steveklabnik:gh28851, r=alexcrichton

If you try to put something that's bigger than a char into a char
literal, you get an error:

    fn main() {
        let c = 'ஶ்ரீ';
    }

    error: unterminated character constant:

This is a very compiler-centric message. Yes, it's technically
'unterminated', but that's not what you, the user did wrong.

Instead, this commit changes it to

    error: character literal that's larger than a char:

As this actually tells you what went wrong.

Fixes #28851

Fixes as per @brson's review

Improve error message for char literals

If you try to put something that's bigger than a char into a char
literal, you get an error:

    fn main() {
        let c = 'ஶ்ரீ';
    }

    error: unterminated character constant:

This is a very compiler-centric message. Yes, it's technically
'unterminated', but that's not what you, the user did wrong.

Instead, this commit changes it to

    error: character literal may only contain one codepoint

As this actually tells you what went wrong.

Fixes #28851

Auto merge of #29581 - gereeter:unwrap-defmap-refcell, r=nrc

This is basically a more conservative recreation of #24096.

Deprecate `_ms` functions that predate the `Duration` API

Remove use of RefCell<DefMap> in the simpler parts of pat_util

Remove use of RefCell<DefMap> in resolve_lifetime

Remove use of RefCell<DefMap> in check_static_recursion

Unwrap the RefCell around DefMap

More relnotes tweaks

Auto merge of #29543 - Amanieu:asm_mem_constraint, r=alexcrichton

The "m" memory constraint in inline assembly is broken (generates incorrect code or triggers LLVM asserts) and should not be used. Instead, indirect memory operands should be used with "\*m", "=\*m" and "+\*m".

Clang does this transparently by transforming "m" constraints into "\*m" indirect constraints, but for now just being able to use "\*m" directly is enough since asm! isn't stable.

While "\*m" works fine as an input operand, "=\*m" and "+\*m" need to be specified as input operands because they take a pointer value as an input. This PR relaxes the constraint checker to allow constraints starting with "=" or "+" if the constraint string contains a "\*", which indicates an indirect operand.

This (indirectly) fixes these issues: #29382, #16383 and #13366. The code will need to be changed to use "\*m" instead of "m".

Introduce a `SwitchInt` and restructure pattern matching to collect
integers and characters into one master switch.

Auto merge of #29536 - sanxiyn:unused-assign-arg, r=alexcrichton

Fix #29136.

[MIR-trans] Fix handling of non-alloca temps in trans_operand_into()