std: Fix process spawn for arguments ending in backslashes on Windows

Fix `make_command_line` for the case of backslashes at the end of an
argument requiring quotes.  We must encode the command and arguments
such that `CommandLineToArgvW` recovers them in the spawned process.
Simplify the logic by using a running count of backslashes as they
are encountered instead of looking ahead for quotes following them.

Extend `test_make_command_line` to additionally cover:

* a leading quote in an argument that requires quotes,
* a backslash before a quote in an argument that requires quotes,
* a backslash at the end of an argument that requires quotes, and
* a backslash at the end of an argument that does not require quotes.

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

- Successful merges: #24649, #24806, #24809, #24811
- Manual merges: #24812

Added a link address to Vectors page of the book

At https://doc.rust-lang.org/book/vectors.html, there should be a link to
Generics page but the link address is ommitted and thus link is not functioning
well. So I added a link definition to the vectors.md.

r? @steveklabnik

add import (fixup #24649)

Rollup merge of #24812 - jest:master, r=steveklabnik

Conflicts:
src/doc/trpl/variable-bindings.md

Rollup merge of #24811 - rkruppe:reference-audit, r=steveklabnik

Mention `non_ascii_idents` feature gate and remove unused productions.

r? @steveklabnik

Rollup merge of #24809 - conradkleinespel:master, r=Manishearth

The reference has broken links. This should fix it.

Rollup merge of #24806 - FuGangqiang:doc, r=Manishearth

Rollup merge of #24649 - nham:path_new_examples, r=steveklabnik

A number of spell-checking corrections.

Auto merge of #24803 - SkylerLipthay:patch-2, r=huonw

The description of the syntax for single byte literals is missing the preceding `b` distinction.

Audit reference manual: 3.2 Special unicode productions

Mention non_ascii_idents feature gate and remove unused productions

librustc_trans: Don't ICE on unsized type behind raw pointer in nullable pointer opt.

fix compiler plugins path in doc/reference.md

Auto merge of #24783 - jooert:unittestguidelines, r=alexcrichton

Changes the style guidelines regarding unit tests to recommend using a sub-module named "tests" instead of "test" for unit tests as "test" might clash with imports of libtest (see #23870, #24030 and http://users.rust-lang.org/t/guidelines-naming-of-unit-test-module/1078 for previous discussions).

r? @alexcrichton

fix doc

Reference manual 3.5.2.3.1

The description of the syntax for single byte literals is missing the preceding `b` distinction.

Auto merge of #24724 - alexcrichton:symlink-stable, r=aturon

These functions were intended to be introduced as `#[stable]` as a stable API
was deprecated in favor of them, but they just erroneously forgot the stability
attributes.

Remove obsolete "MarkerTrait" from tests.

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

- Successful merges: #24662, #24722, #24725, #24729, #24736, #24749, #24751, #24766, #24769, #24772, #24775, #24790
- Failed merges: #24760

Fix #20616

More small syntax changes in reference.md

Rollup merge of #24790 - jooert:fix24774, r=steveklabnik

Fixes #24774.

r? @steveklabnik

Rollup merge of #24775 - mbrubeck:reference, r=steveklabnik

Update 7.2.20 (`for` expressions):

* `for` loops now use `IntoIterator` instead of just `Iterator`
* Simplify the example by removing unnecessary `Vec::iter` call.

...and a fix for a minor formatting error.

r? @steveklabnik

Rollup merge of #24772 - steveklabnik:gh24712, r=alexcrichton

FIxes #24712

Rollup merge of #24769 - SkylerLipthay:patch-1, r=steveklabnik

`is` and `us` suffixes are deprecated in favor of `isize` and `usize`.

Rollup merge of #24766 - nwin:patch-1, r=steveklabnik

Changed in #22838.

audited (raw) byte string literals @ #16676

Rollup merge of #24751 - lstat:feature-gate-22820-dups, r=brson

As part of the audit for #22820 the following duplicate feature
gate tests were removed:

* `box_patterns`
* `simd_ffi`

These tests for `box_patterns` and `simd_ffi` were added in #23578,
however there were existing tests in #20723 and #21233 respectively.

r? @nrc

Rollup merge of #24749 - lstat:feature-gate-22820, r=nrc

As part of the audit for #22820 the following feature gate tests have been
added:

* `negate_unsigned`
* `on_unimplemented`
* `optin_builtin_traits`
* `plugin`
* `rustc_attrs`
* `rustc_diagnostic_macros`
* `slice_patterns`

In addition some feature gate error message typos fixed.

Rollup merge of #24736 - steveklabnik:doc_mutability, r=alexcrichton

Okay, last chapter other than ownership stuff. :confetti_ball:

Rollup merge of #24729 - mdinger:reference_grammar, r=steveklabnik

From https://github.com/rust-lang/rust/issues/24723#issuecomment-95636827 :

> Yes, I've been meaning to just remove it in favor of GRAMMAR.md
> - steveklabnik

Rollup merge of #24725 - steveklabnik:doc_operators_and_overloading, r=alexcrichton

I forgot these heavily use associated types, so move it after that
as well.

Rollup merge of #24722 - steveklabnik:doc_deref, r=alexcrichton

r? @alexcrichton

Rollup merge of #24662 - steveklabnik:gh24656, r=steveklabnik

Fixes #24656

r? @pnkfelix

I just added the examples, but if the wording needs expanded too, let me know what you think should be added :)

In [-] doc buttons, change hyphen ‘-’ to minus ‘−’

The minus sign ‘−’ is the same width as the plus sign ‘+’, so the button’s transition between the two symbols will look more smooth.

Whoops, please tidy

rustdoc: change color of trait types

Fixes #24441

TRPL: deref coercions

Add examples by @pnkfelix to fmt precision

Fixes #24656

rustbook: Fixes display of navigation links in README.html.

TRPL: mutability

TRPL: operators and overloading

I forgot these heavily use associated types, so move it after that
as well.

Auto merge of #24553 - nikomatsakis:issue-22779-overconstrained-impl, r=pnkfelix

Rather than storing the relations between free-regions in a global
table, introduce a `FreeRegionMap` data structure. regionck computes the
`FreeRegionMap` for each fn and stores the result into the tcx so that
borrowck can use it (this could perhaps be refactored to have borrowck
recompute the map, but it's a bid tedious to recompute due to the
interaction of closures and free fns). The main reason to do this is
because of #22779 -- using a global table was incorrect because when
validating impl method signatures, we want to use the free region
relationships from the *trait*, not the impl.

Fixes #22779.

Change name of unit test sub-module to "tests".

Changes the style guidelines regarding unit tests to recommend using a
sub-module named "tests" instead of "test" for unit tests as "test"
might clash with imports of libtest.

fix rustc_driver tests

Remove keywords from reference because they're already in grammar

Updates to grammar

Remove ebnf from reference

Split up tests, reduce coverage

Auto merge of #24594 - doomsplayer:patch-2, r=alexcrichton

why use dummy implementation on linux?

Deduplicate slice iter offset/transmute code.

[reference] Update 7.2.20: For expressions.

* `for` loops now use `IntoIterator` instead of just `Iterator`
* Simplify the example by removing unnecessary `Vec::iter` call.

[reference] Fix missing formatting.

Auto merge of #24744 - bluss:reference-1, r=steveklabnik

Audit & Edit Chapter 8.1 Types in reference manual

- Remove mention of unit type
- Update closure types and reference types sections
- Fix minor typos

Remove reference to 'to'

FIxes #24712

reference: Audit & Edit chapter 8.1 Types.

- Remove mention of unit type
- Update closure types and reference types sections
- Fix minor typos

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

- Successful merges: #24523, #24698, #24699, #24700, #24706, #24717, #24718, #24721, #24727
- Failed merges:

Rollup merge of #24727 - rkruppe:reference-audit, r=steveklabnik

It was in pretty good shape, but since that is my pet peeve, I clarified the compiler/interpreter distinction and why it is irrelevant for this section. Otherwise only a couple of minor clarifications, and weasel words where reality is more complicated than the text accounted for (e.g., there is more than one kind of library).

r? @steveklabnik

Rollup merge of #24721 - vosen:patch-1, r=steveklabnik

`us` and `is` were replaced with `usize` and `isize` some time ago. Other than that, 3.5.2.1.5 is correct.

Rollup merge of #24717 - liigo:add-back-toggle-links, r=alexcrichton

r? @alexcrichton (since you added `.stability` warning messages)

Rollup merge of #24706 - tamird:remove-DST-comment, r=alexcrichton

`ToCStr` was removed with `old_io` and the current method `as_os_str`
is inherent to `Path`, meaning there is no suitable trait bound that
could be used here.

r? @alexcrichton

Rollup merge of #24700 - rkruppe:doc-typos, r=alexcrichton

r? @steveklabnik

Rollup merge of #24699 - mbrubeck:doc-edit, r=steveklabnik

r? @steveklabnik

Rollup merge of #24698 - steveklabnik:remove_debug_display, r=steveklabnik

this is too small for its own thing, I think.

Rollup merge of #24523 - GuillaumeGomez:clean-error-codes, r=Manishearth

Reference manual 3.5.2.1.5

`is` and `us` suffixes are deprecated in favor of `isize` and `usize`.

Byte string literals are now fixed-size arrays

Changed in #22838.

audited (raw) byte string literals @ #16676

Remove direct variant-dispatch entirely from trans_struct_drop.

This addresses to-do in my code, and simplifies this method a lot to boot.

(The necessary enum dispatch has now effectively been shifted entirely
into the scheduled cleanup code for the enum contents.)

regression test.

Added new kind of drop-glue that just drops the type's contents,
without invoking the Drop::drop implementation.

This is necessary for dealing with an enum that switches own `self` to
a different variant while running its destructor.

Fix #23611.

Latent bug in iter_structural_ty: handle `_match::Single` on zero-variant enum.

(This may not be the *best* fix, compared to e.g. returning
`_match::NoBranch` from `trans_switch` on a zero-variant enum. But it
is one of the *simplest* fixes available.)

Auto merge of #24759 - Manishearth:rollup2, r=Manishearth

r? @Manishearth

fixup #24754

Rollup merge of #24754 - iliekturtles:patch-1, r=steveklabnik

Rollup merge of #24753 - tynopex:patch-1, r=steveklabnik

 Add section for range expressions.

Rollup merge of #24752 - doomrobo:patch-1, r=steveklabnik

 Updated sample code to updated syntax (now compiles). Also tweaked the text to reflect the change.

Rollup merge of #24750 - withoutboats:patch-1, r=steveklabnik

 1) Moved 'while' section below 'loop', 'break', and 'continue'; 2) Added information to 'while' and 'for' loops that they interact with 'break' and 'continue' and may have a lifetime label. 3) Clarified labeling syntax on the infinite loops.

Rollup merge of #24743 - geofft:trpl-macros-links, r=steveklabnik

 r? @steveklabnik

Rollup merge of #24742 - mbrubeck:keywords, r=steveklabnik

 * Add two missing keywords, `Self` and `proc`
* Fix some mis-alphabetized keywords

r? @steveklabnik

Rollup merge of #24740 - mbrubeck:reference, r=steveklabnik

 r? @steveklabnik

Rollup merge of #24739 - nrc:span-item, r=huonw

 (And other kinds of items).

This brings trait/impl items in line with top-level items in this respect. It means we don't get the attributes when showing error messages for methods, etc. (a good thing, I think). Since attributes have spans it is still easy to create a span for the item + all attributes.

Rollup merge of #24734 - vadimcn:patch-1, r=alexcrichton

 'win32' -> 'windows', added 'ios'

Rollup merge of #24733 - nwin:patch-1, r=steveklabnik

 Consistency. The book also refers to it as trait objects.

Feature-gate associated constants.

Get associated consts working in match patterns.

Fix dead code check for associated const impls.

Add previously omitted associated const tests.

Fix a number of bugs that interfered with cross-crate usage of associated consts.

Functional changes for associated constants. Cross-crate usage of associated constants is not yet working.

Structural changes for associated constants

Introduces new variants and types in syntax::ast, middle::ty, and middle::def.

Fix spelling errors.

Update reference.md

Add section for range expressions.

Update "Generic Functions" section

Updated sample code to updated syntax (now compiles). Also tweaked the text to reflect the change.

Remove duplicate feature gate tests

As part of the audit for #22820 the following duplicate feature
gate tests were removed:

* `box_patterns`
* `simd_ffi`

These tests for `box_patterns` and `simd_ffi` were added in #23578,
however there were existing tests in #20723 and #21233 respectively.

Add feature gate tests

As part of the audit for #22820 the following feature gate tests have
been added:

* `rustc_diagnostic_macros`

/cc #19624

Add feature gate tests

As part of the audit for #22820 the following feature gate tests have
been added:

* `negate_unsigned`
* `on_unimplemented`
* `optin_builtin_traits`
* `plugin`
* `rustc_attrs`
* `slice_patterns`

Add ')' in `advaced_slice_patterns` error message

Add missing '\' for `custom_derive` error message

Fixed typo

Improve information about loops

1) Moved 'while' section below 'loop', 'break', and 'continue'; 2) Added information to 'while' and 'for' loops that they interact with 'break' and 'continue' and may have a lifetime label. 3) Clarified labeling syntax on the infinite loops.