Rollup merge of #21373 - angst7:pointer_doc_1, r=steveklabnik

Updated incorrect error messages, and removed explicit return statements from example code.

Rollup merge of #21325 - nagisa:thread-panicking, r=steveklabnik

Previous wording wasn’t clear about its actual behaviour. It could be
interpreted as answering either:

* Can current thread panic?
* Is current thread unwinding because of panic?

r? @steveklabnik

Rollup merge of #21217 - Gankro:docadoca, r=steveklabnik

Not sure on what *exactly* should be said here, but I think this is the most important bit. This PR also establishes conventions for describing performance minimally.

I suggest to describe preformance for individual methods we use a `# Performance` heading. Not sure if we should have

```
# Performance: O(1)
details details
```
or

```
# Performance:
O(1)
details details
```

Since I think most methods don't need discussion, the former seems more resonable. But it's kind of weird to have info "in" the heading.

r? @steveklabnik

Rollup merge of #21091 - steveklabnik:add_attribute_docs, r=huonw

Fixes #6444

I need some help with the macro docs here, as I couldn't figure it out. @kmcallister @eddyb  or someone else?

Rollup merge of #21056 - steveklabnik:return_strings, r=sfackler

When we moved over to the book, we lost this.

Document remaining attributes and macros

Fixes #6444

Return the String guide to its former glory.

When we moved over to the book, we lost this.

Auto merge of #21078 - js-ojus:master, r=steveklabnik

* Include an illustration of a function that accepts two closures.

Auto merge of #21187 - oli-obk:feature/hint_struct_field_access, r=alexcrichton

rebase and fix of #19267

Auto merge of #21473 - alexcrichton:rollup, r=alexcrichton

Make diagnostic ordering deterministic

Revert "Use assume to inform the optimiser about refcount invariants"

This reverts commit a729a404945de10f99e2530a5c28952996532b29.

Revert "Add assumptions that the pointer is non-null"

This reverts commit 9bbfd681c9fa47f462a89e8f5eedd3fa2a5de2e7.

Revert "Add more explanation for why the assumes are there"

This reverts commit a7525bc4c8eb8507a5c248d29286e77133217cf3.

More test fixes and rebase conflicts

rollup merge of #20179: eddyb/blind-items

Conflicts:
src/librustc/diagnostics.rs
src/librustdoc/clean/mod.rs
src/librustdoc/html/format.rs
src/libsyntax/parse/parser.rs

rollup merge of #21258: aturon/stab-3-index

Conflicts:
src/libcore/ops.rs
src/librustc_typeck/astconv.rs
src/libstd/io/mem.rs
src/libsyntax/parse/lexer/mod.rs

rollup merge of #21252: nikomatsakis/assoc-type-ice-hunt-take-2

Project region bounds out of the trait when deciding whether a projection type outlives a given regions.

Fixes #20890.
Fixes #21150.

rollup merge of #20642: michaelwoerister/sane-source-locations-pt1

Conflicts:
src/librustc_trans/trans/debuginfo.rs

Move regression test for #20971 into run-fail, since it panics.

Fix type inference problems in tests and docs

Test fixes and rebase conflicts

Deny imports after non-item statements.

Added another test with success local-item shadowing

Added some tests for arbitrary ordered view items

tests: fix fallout of merging ast::ViewItem into ast::Item.

rollup merge of #21463: sanxiyn/demut

rollup merge of #21462: ahmedcharles/remove-ratchet

Conflicts:
src/libtest/lib.rs

rollup merge of #21457: alexcrichton/issue-21436

Conflicts:
src/liballoc/boxed.rs
src/librustc/middle/traits/error_reporting.rs
src/libstd/sync/mpsc/mod.rs

rollup merge of #21447: cmr/master

Closes #13971

rollup merge of #21446: stepancheg/boxed-test

Conflicts:
src/liballoc/boxed.rs

rollup merge of #21444: petrochenkov/null

Conflicts:
src/libstd/sync/mpsc/select.rs

rollup merge of #21441: alexcrichton/rustc-opts

This is a bit of cleanup work to clean out some old deprecated flags and deprecated lint names from the compiler (they've been deprecated for quite awhile now).

This also notably puts `--pretty` behind the `-Z unstable-options` flag (where it was supposed to be previously).

rollup merge of #21438: taralx/kill-racycell

Conflicts:
src/libstd/sync/mpsc/mod.rs

rollup merge of #21437: FlaPer87/snapshot

r? @alexcrichton

rollup merge of #21433: alfie/typobook

Tiny fix

rollup merge of #21429: GuillaumeGomez/macro-fix

This is little clean code of this PR: #21366. I patched the same thing as aochagavia but too slowly obviously. This is a merge of our two codes, more "rust-like".

rollup merge of #21423: oli-obk/prettier_read_until

Conflicts:
src/libstd/io/mod.rs

rollup merge of #21421: huonw/one-suggestion-per-trait

This is clearly useless, the user doesn't need to know that they could
implement/import `foo::bar::Baz` 4 times.

Fixes #21405.

rollup merge of #21419: Toby-S/patch-1

This just corrects a couple of typos in doc comments, and changes some to conform to the Rust guidelines.

rollup merge of #21418: Aatch/assume-refcount

The reference count can never be 0, unless we're about to drop the data
completely. Using the `assume` intrinsic allows us to inform LLVM about
that invariant, meaning it can avoid unnecessary drops.

---

Before and after IR: https://gist.github.com/Aatch/3786d20df2edaad6a0e8

Generated from the example in #13018

Fixes #13018

rollup merge of #21414: ejjeong/aarch64-linux-android

Initial support for aarch64-linux-android (#18920)
- Add new configuration files
- Modify some options to compile & link succesfully.
  (PIE, disable tls on jemalloc, modify some external function linkage, ..)
- To build, refer to https://github.com/rust-lang/rust/wiki/Doc-building-for-android.
   (tested with platform=21 and toolchain=aarch64-linux-android-4.9)

rollup merge of #21413: ahmedcharles/remove-test-features

This isn't the entire set of changes, there are more coming. #19145

rollup merge of #21411: P1start/help-tweaks

Conflicts:
src/librustc_typeck/check/closure.rs

rollup merge of #21396: japaric/no-parens-in-range

Conflicts:
src/libsyntax/parse/lexer/comments.rs

rollup merge of #21394: japaric/nonono

r? @FlaPer87

rollup merge of #21393: loganchien/fix-if-stmt-doc-title

Slightly change the title to make it look more consistent with other chapters (e.g. Match.)

rollup merge of #21392: japaric/iter

closes #20953
closes #21361

---

In the future, we will likely derive these `impl`s via syntax extensions or using compiler magic (see #20617). For the time being we can use these manual `impl`s.

r? @aturon
cc @burntsushi @Kroisse

rollup merge of #21391: klutzy/rt-time-cleanup

They are unused since libtime is gone.

cc #20861

rollup merge of #21389: retep998/timer

Fixes #20943 and adds a test for it
r? @alexcrichton

rollup merge of #21372: arielb1/remove-the-box

It is not used anymore

rollup merge of #21368: tomjakubowski/rustdoc-miscellany

Conflicts:
src/librustdoc/clean/mod.rs

rollup merge of #21367: steveklabnik/remove_gate

This gate was `Accepted`, so we shouldn't need these.

rollup merge of #21354: eddyb/vec-ufcs

There are two limitations to the macro that this addresses:
1. the expected type is not propagated, coercions don't trigger
2. references inside element expressions don't outlive the `Vec`

Both of these limitations are caused by the block in the
macro expansion, previously needed to trigger a coercion
from `Box<[T; N]>` to `Box<[T]>`, now possible with UFCS.

rollup merge of #21342: Diggsey/issue-21310

Fixes #21310

rollup merge of #21340: pshc/libsyntax-no-more-ints

Collaboration with @rylev!

I didn't change `int` in the [quasi-quoter](https://github.com/pshc/rust/blob/99ae1a30f3ca28c0f7e431620560d30e44627124/src/libsyntax/ext/quote.rs#L328), because I'm not sure if there will be adverse effects.

Addresses #21095.

rollup merge of #21333: stepancheg/trans-write-diag

File cannot be written, for example, if directory does not exist.

Before this commit:

```
% rustc -o nonexistent/program program.rs
error: could not write output: No such file or directory
```

With this commit:

```
% rustc -o nonexistent/program program.rs
error: could not write output to nonexistent/program.0.o: No such file or directory
```

This is useful when full rust command is not displayed, or when last error is followed by thousands of warnings.

rollup merge of #21289: brson/errorcodes

This does the bare minimum to make registration of error codes work again. After this patch, every call to `span_err!` with an error code gets that error code validated against a list in that crate and a new tidy script `errorck.py` validates that no error codes are duplicated globally.

There are further improvements to be made yet, detailed in #19624.

r? @nikomatsakis

rollup merge of #21274: estsauver/21270

The timezone of the server that builds rustc nightlies
appears to be in UTC. From what I can tell, it builds
the nightlies at 0300 UTC, which takes about ~15 minutes.
So, there were a couple options here for which offset to use.

UTC+3 would ensure that you always got the latest rust, but it
would mean the script is broken ~15/20 minutes a day. UTC+4
means users get a stale nightly for 45 minutes, but that feels
okay to me.

Ideally, buildbot would publish the "current" nightly, but
that seems unneeded relative to fixing it for all Timezones
quickly.

Fixes #21270

rollup merge of #21157: steveklabnik/deprecation

rollup merge of #21138: mkpankov/master

When combined with '--save' and '--date', it uses previously saved
tarball, making possible to re-install in offline mode.
r?

rollup merge of #21053: apasel422/exact

rollup merge of #19913: KOMON/rust-mode-emacs-indentation

I added an option to auto-indent method chains to line up along their '.' operators. Like so:

```
let input = io::stdin().readline()
                       .ok()
                       .expect("Failed to read line");
```

The old default would indent like so:
```
let input = io::stdin().readme()
    .ok()
    .expect("Failed to read line");
```

The Rust guide explicitly condones the former, so I thought it would be nice for the emacs mode to support it. It's off by default, you have to set ```rust-indent-method-chain``` to ```t``` via your .emacs or the customize menu

Improve quality of String#to_slice() deprecation message

Change init-large-type to use child thread

Fallout from stabilization.

Deprecate slicing methods in favor of notation

This commit deprecates `slice`, `slice_from`, `slice_to` and their
mutable variants in favor of slice notation.

The `as_slice` methods are left intact, for now.

[breaking-change]

Stabilize Index traits and most range notation

This commit marks as `#[stable]`:

* The `Index` and `IndexMut` traits. These are stabilized as taking the
  index itself *by reference*; after extensive discussion it was
  determined that this is a better match with our choices
  elsewhere (e.g. making comparison operators auto-reference), and that
  the use cases for by-value indices are better handled through
  `IndexSet`.

* The `Range`, `RangeFrom` and `RangeTo` structs, introduced for range
  notation.

* Various impls of `Index` and `IndexMut`.

The `FullRange` struct is left unstable as we may wish to rename it to
`RangeFull` in the future.

This commit also *removes* the `Step` trait in favor of direct
implementation of iterator traits on ranges for integers. The `Step`
trait was not a terribly useful factoring internally, and it is likely
that external integer types are best off implementing range iterators
directly. It was removed to simplify the API surface. We can always
reintroduce `Step` later if it turns out to be useful.

Due to this removal, this is a:

[breaking-change]

docs: fix fallout of merging ast::ViewItem into ast::Item.

rustdoc: fix fallout of merging ast::ViewItem into ast::Item.

rustc_resolve: fix fallout of merging ast::ViewItem into ast::Item.

rustc_driver: fix fallout of merging ast::ViewItem into ast::Item.

rustc_trans: fix fallout of merging ast::ViewItem into ast::Item.

rustc_typeck: fix fallout of merging ast::ViewItem into ast::Item.

rustc: fix fallout of merging ast::ViewItem into ast::Item.

rustc_back: fix fallout of merging ast::ViewItem into ast::Item.

syntax: fix fallout of merging ast::ViewItem into ast::Item.

syntax: merge ast::ViewItem into ast::Item.

Remove feature(import_shadowing).

De-mut the parser

Auto merge of #21242 - richo:no-perl, r=brson

There's only one build-critical path in which perl is used, and it was to do a text replacement trivially achievable with sed(1).

I ported the indenter script because it [appears to be used][indenter], but removed check links because it appears to be entirely out of date.

[indenter]: https://github.com/rust-lang/rust/blob/master/src/librustc/util/common.rs#L60-70

debuginfo: Make debuginfo source location assignment more stable (Pt. 1)

So far, the source location an LLVM instruction was linked to was controlled by
`debuginfo::set_source_location()` and `debuginfo::clear_source_location()`.
This interface mimicked how LLVM's `IRBuilder` handles debug location
assignment. While this interface has some theoretical performance benefits, it
also makes things terribly unstable: One sets some quasi-global state and then
hopes that it is still correct when a given instruction is emitted---an
assumption that has been proven to not hold a bit too often.

This patch requires the debug source location to be passed to the actual
instruction emitting function. This makes source location assignment explicit
and will prevent future changes to `trans` from accidentally breaking things in
the majority of cases.

This patch does not yet implement the new principle for all instruction kinds
but the stepping experience should have improved significantly nonetheless
already.

Auto merge of #21227 - sellibitze:core-ops-for-references, r=aturon

As discussed with @aturon I added implementations of various op traits for references to built-in types which was already suggested by the ops reform RFC.

The 2nd commit updates the module documentation of core::ops to fully reflect the recent change from pass-by-reference to pass-by-value and expands on the implications for generic code.

Remove ratchet().

Remove compare_to_old().

Remove write_metric_diff and supporting code.

std: Rename Show/String to Debug/Display

This commit is an implementation of [RFC 565][rfc] which is a stabilization of
the `std::fmt` module and the implementations of various formatting traits.
Specifically, the following changes were performed:

[rfc]: https://github.com/rust-lang/rfcs/blob/master/text/0565-show-string-guidelines.md

* The `Show` trait is now deprecated, it was renamed to `Debug`
* The `String` trait is now deprecated, it was renamed to `Display`
* Many `Debug` and `Display` implementations were audited in accordance with the
  RFC and audited implementations now have the `#[stable]` attribute
  * Integers and floats no longer print a suffix
  * Smart pointers no longer print details that they are a smart pointer
  * Paths with `Debug` are now quoted and escape characters
* The `unwrap` methods on `Result` now require `Display` instead of `Debug`
* The `Error` trait no longer has a `detail` method and now requires that
  `Display` must be implemented. With the loss of `String`, this has moved into
  libcore.
* `impl<E: Error> FromError<E> for Box<Error>` now exists
* `derive(Show)` has been renamed to `derive(Debug)`. This is not currently
  warned about due to warnings being emitted on stage1+

While backwards compatibility is attempted to be maintained with a blanket
implementation of `Display` for the old `String` trait (and the same for
`Show`/`Debug`) this is still a breaking change due to primitives no longer
implementing `String` as well as modifications such as `unwrap` and the `Error`
trait. Most code is fairly straightforward to update with a rename or tweaks of
method calls.

[breaking-change]
Closes #21436

Auto merge of #21395 - sfackler:fix-cvar-test, r=alexcrichton

r? @alexcrichton

remove `NoSendItem` and `NoSyncItem`

Display negative trait implementations correctly in rustdoc

Added doc test

Auto merge of #21065 - ColonelJ:master, r=brson

Removed use of unused LDPATH variable on Windows as is done for other platforms, and added GCC flag to ensure MINGW's ANSI compatible STDIO functions are used wherever available (required by jemalloc).

Without these changes it ends up setting the PATH twice, and the second time the PATH begins with `:` which is invalid.  Also the regular msvcrt printf-like functions would be used which don't understand stuff like %hhd and %z which jemalloc uses.

This change ought not to make any difference to the output but it fixes the build process for me since at least my build environment couldn't handle that broken path caused by LDPATH being empty.

implement `ExactSizeIterator` for `slice::{Chunks,ChunksMut}`

tests: uncomment regression tests for 13970, 13971, 13972

Closes #13971

also forward Iterator::size_hint()

Auto merge of #21439 - alexcrichton:rollup, r=alexcrichton

Continuation of https://github.com/rust-lang/rust/pull/21428

Make multiline errors work with codes

Changed rust-mode code for tabs -> spaces

add stability text like suggested in discussion

Register snapshot for 9006c3c

alloc::boxed: enable test

Previously test was disabled due to `#[cfg(test)]` before `mod boxed`.