Rollup merge of #29068 - apasel422:tidy, r=alexcrichton

r? @alexcrichton

Rollup merge of #29066 - dcarral:fix_issue_29063, r=steveklabnik

Regarding [#29063 _[Docs] Terminology inconsistency between 'iterator adapters' and 'iterator adaptors'_](https://github.com/rust-lang/rust/issues/29063) :

This PR replaces 'iterator adapters' appearances (in TRPL book) to 'iterator adaptors', thus embracing the terminology used along the API docs and achieving consistency between both sources.

Rollup merge of #29062 - rgardner:rgardner-fix-book-comp-warning, r=alexcrichton

Before this commit, the first "A Rust library" code sample produced
the following compilation warning:
```
test.rs:7:22: 7:36 warning: unnecessary parentheses around `for` head
expression, #[warn(unused_parens)] on by default
test.rs:7             for _ in (0..5_000_000) {
```

This commit just removes the parens around the range 0..5_000_000 thereby removing the compilation warning.

Rollup merge of #29060 - tshepang:consistency, r=nikomatsakis

Rollup merge of #29059 - tshepang:typo, r=nikomatsakis

Rollup merge of #29058 - tshepang:rename, r=steveklabnik

Shoud have been part of commit 0b13ee0ced39

Rollup merge of #29047 - gkoz:doc_typo, r=steveklabnik

None

Rollup merge of #29022 - apasel422:spell, r=steveklabnik

r? @steveklabnik

Rollup merge of #28906 - tshepang:link, r=nikomatsakis

Auto merge of #27017 - luqmana:25594-sysv-abi-ffi, r=bkoropoff

Fixes #25594.

Update the comments for Win64 ABI in tests.

rust_trans: struct argument over ffi were passed incorrectly in some situations on x86_64.

Auto merge of #28980 - nrc:unsafe-macros, r=@pnkfelix

This is a [breaking change].

@brson could you do a Crater run with this PR please?

Lets not land till Crater says its OK.

This was discussed at https://internals.rust-lang.org/t/does-anyone-use-the-push-pop-unsafe-macros/2702

Remove unnecessary parentheses around range expressions

Auto merge of #29012 - tari:issue-28676, r=luqmana

Fixes #28676.

There doesn't seem to be a good way to add a test for this, but I tested the repro in #28676 and confirmed it now yields the correct result.

Replace 'adapters' to 'adaptors' in TRPL book

Regarding #29063: Replace 'iterator adapters' appearances to
'iterator adaptors', thus embracing the terminology used along the
API docs and achieving consistency between both sources.

book: be consistent with preceding example

book: fix typo

reference: 'struct' is more common that 'structure'

Shoud have been part of commit 0b13ee0ced39

reference: add link to the symbols

Resolve unused_parens compilation warning

Before this commit, the first "A Rust library" code sample produced
the following compilation warning:
```
test.rs:7:22: 7:36 warning: unnecessary parentheses around `for` head
expression, #[warn(unused_parens)] on by default
test.rs:7             for _ in (0..5_000_000) {
```

This commit just removes the parens around the range 0..5_000_000.

Add test case for #28676.

Auto merge of #29044 - Ms2ger:cleanup-log, r=alexcrichton

Auto merge of #29003 - tari:freebsd-current_exe-vec, r=alexcrichton

Fixes #28995.

Auto merge of #29001 - arielb1:normalized-foreign, r=eddyb

This is needed as item types are allowed to be unnormalized.

Fixes an ICE that occurs when foreign function signatures contained
an associated type.

Fixes #28983

Fix the link to `Default` trait in the prelude docs

Auto merge of #28909 - GuillaumeGomez:patch-3, r=pnkfelix

r? @Manishearth

Remove the unnecessary local variable in set_logger.

Use if-let rather than map() in parse_logging_spec.

Auto merge of #28827 - thepowersgang:unsafe-const-fn-2, r=Aatch

This is the original test implementation, which works according to the tests I wrote, but might need a review.

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

- Successful merges: #28991, #29004, #29006, #29013, #29016, #29024, #29027, #29028, #29029, #29032, #29035
- Failed merges:

fix tidy

Rollup merge of #29035 - rutsky:patch-6, r=steveklabnik

r? @steveklabnik

The link is broken here: <https://doc.rust-lang.org/std/io/#types>.

Looks like crate documentation generator uses only first paragraph of the module documentation and
so doesn't resolve the link defined below.

Probably this behaviour of the documentation generator should be considered as a bug and should be reported.

Rollup merge of #29032 - goyox86:goyox86/rusfmting-librustc_bitflags, r=alexcrichton

Hi Rustaceans!

This is the result of running latest rustfmt on librustc_bitflags!

//cc @nrc

Rollup merge of #29029 - steveklabnik:iter_function_docs, r=alexcrichton

Flesh the docs out with examples.

Rollup merge of #29028 - Seeker14491:patch-1, r=Manishearth

Having this code section hidden is misleading because it makes it look like implementing Circle for Foo automatically makes Foo implement Shape.

Rollup merge of #29024 - mseri:patch-1, r=nrc

r? @nrc

Rollup merge of #29016 - sanxiyn:data-layout, r=alexcrichton

Rollup merge of #29013 - chrisccerami:fix_broken_lifetime_elision_link, r=alexcrichton

This link was added in #28842 but doesn't work at https://doc.rust-lang.org/nightly/book/lifetimes.html. What works in my markdown preview doesn't work live, and vice versa.

Rollup merge of #29006 - arielb1:callee-outlives-call, r=pnkfelix

This rather crucial requirement was not checked. In most cases, that
didn't cause any trouble because the argument types are required to
outlive the call and are subtypes of a subformula of the callee type.

However, binary ops are taken by ref only indirectly, without it being
marked in the argument types, which led to the argument types not being
constrained anywhere causing spurious errors (as these are basically
unconstrainable, I don't think this change can break code). Of course,
the old way was also incorrent with contravariance, but that is still
unsound for other reasons.

This also improves rustc::front to get RUST_LOG to *somewhat* work.

Fixes #28999. That issue is one of the several regression introduced by #28669.

r? @pnkfelix

Rollup merge of #29004 - frewsxcv:regression-test-22814, r=alexcrichton

Closes #22814

Rollup merge of #28991 - goyox86:goyox86/rustfmting-liblog-II, r=alexcrichton

Hi Rustaceans!

This is the second version of running rustfmt on liblog https://github.com/rust-lang/rust/pull/28898. There are only rustfmt suggested changes. Because I think that some patterns here (that I fixed up in the prev PR need to be discussed in detail.

//cc @nrc

Auto merge of #28816 - petrochenkov:unistruct, r=nrc

This patch uses the same data structures for structs and enum variants in AST and HIR. These changes in data structures lead to noticeable simplification in most of code dealing with them.
I didn't touch the top level, i.e. `ItemStruct` is still `ItemStruct` and not `ItemEnum` with one variant, like in the type checker.
As part of this patch, structures and variants get the `kind` field making distinction between "normal" structs, tuple structs and unit structs explicit instead of relying on the number of fields and presence of constructor `NodeId`. In particular, we can now distinguish empty tuple structs from unit structs, which was impossible before! Comprehensive tests for empty structs are added and some improvements to empty struct feature gates are made. Some tests don't pass due to issue https://github.com/rust-lang/rust/issues/28692 , they are still there for completeness, but are commented out.
This patch fixes issue mentioned in https://github.com/rust-lang/rust/issues/16819#issuecomment-139509861, now emptiness of tuple structs is checked after expansion.
It also touches https://github.com/rust-lang/rust/issues/28750 by providing span for visit_struct_def
cc https://github.com/rust-lang/rust/pull/28336

r? @nrc

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

- Successful merges: #28988, #28989, #28990, #28997, #29007, #29015
- Failed merges: #28906

fix link on std::result::Result

The link is broken here: <https://doc.rust-lang.org/std/io/#types>.

Looks like crate documentation generator uses only first paragraph of the module documentation and
so doesn't resolve the link defined below.

Revert "fixups"

This reverts commit 5b8335ede4214f802eab765ca99b5afa9057973c.

Rustfmt-ing librustc_bitflags.

Document the free functions of std::iter

Flesh the docs out with examples.

Unhide some code from the Traits section

Having this code section hidden is misleading because it makes it look like implementing Circle for Foo automatically makes Foo implement Shape.

Rollup merge of #29015 - dripton:master, r=alexcrichton

Just a single-character typo fix.

Rollup merge of #29007 - dnwade:patch-2, r=alexcrichton

r? @steveklabnik

Rollup merge of #28997 - bezelga:master, r=steveklabnik

Fixing some words in the beginning of the Ownership system chapters.

Rollup merge of #28990 - ahmedcharles:libarena, r=nrc

Rollup merge of #28989 - ahmedcharles:liballoc_system, r=nrc

Rollup merge of #28988 - ahmedcharles:liballoc_jemalloc, r=nrc

Add E0511 error explanation

fixups

rustfmt libfmt_macros

Correct spelling in docs

Comment on the purpose(s) of NodeId in VariantData

Merge VariantData and VariantData_

Fix rebase 2

Merge struct fields and struct kind

Dict -> Struct, StructDef -> VariantData, def -> data

Fix rebase

Fix failing privacy test

Test and gate empty structures and variants better

Provide span for visit_struct_def + remove some dead code

Remove now redundant NodeId from Variant

Decouple structure kinds from NodeIds

Unify structures and enum variants in HIR

Unify structures and enum variants in AST

Manually alligned comments.

Re-running updated rustfmt on liblog.

rustfmt suggested changes.

Avoid using getDataLayout, deprecated in LLVM 3.7

Typo fix

Auto merge of #28993 - frewsxcv:regression-test-17336, r=alexcrichton

Closes #17336

Fix Lifetime Elision link in lifetimes.md

Auto merge of #28987 - ahmedcharles:liballoc, r=brson

typo

win64/msvc: large or oddly-sized types pass by-ref

require a method callee's type to outlive the call

This rather crucial requirement was not checked. In most cases, that
didn't cause any trouble because the argument types are required to
outlive the call and are subtypes of a subformula of the callee type.

However, binary ops are taken by ref only indirectly, without it being
marked in the argument types, which led to the argument types not being
constrained anywhere causing spurious errors (as these are basically
unconstrainable, I don't think this change can break code). Of course,
the old way was also incorrent with contravariance, but that is still
unsound for other reasons.

This also improves rustc::front to get RUST_LOG to *somewhat* work.

Fixes #28999

Auto merge of #28986 - ahmedcharles:error-index-generator, r=brson

Add regression test for #22814

Closes #22814

normalize the types of foreign functions

This is needed as item types are allowed to be unnormalized.

Fixes an ICE that occurs when foreign function signatures contained
an associated type.

Fixes #28983

Fix docs about borrowing and lifetimes

Add regression test for #17336

Closes #17336

Auto merge of #28914 - steveklabnik:doc_iterator, r=alexcrichton

This replaces what was there with a comprehensive overview.

Thanks to @hoverbear for suggesting that these docs needed improvement.

Write better module-level docs for std::iter

This replaces what was there with a comprehensive overview.

Don't use a Vec in os::current_exe on FreeBSD.

Auto merge of #28985 - ahmedcharles:driver, r=nrc

Run rustfmt on error-index-generator.

Run rustfmt on driver.

Run rustfmt on liballoc_system.

Run rustfmt on liballoc_jemalloc.

Run rustfmt on liballoc.

Remove the push_unsafe! and pop_unsafe! macros.

This is a [breaking change].

Run rustfmt on libarena.

Auto merge of #28969 - chrisccerami:link_to_ffi_in_concurrency_chapter, r=steveklabnik