Add regression test for macro expansion

Auto merge of #4575 - Manishearth:suggestions, r=oli-obk

Make more tests rustfixable

Fixes https://github.com/rust-lang/rust-clippy/issues/3630

changelog: Improve suggestions for many lints in preparation for `cargo fix --clippy`

r? @phansch @yaahc

Downgrade op_ref to a MaybeIncorrect suggestion

unnecessary_operation: make test rustfixable

unnecessary_clone: split rustfixable lint out into separate test

string_add, string_add_assign: split tests, make one rustfixable

redundant_static_lifetimes: split test, make rustfixable

renamed_builtin_attr: make test rustfixable

redundant_pattern_matching: make rustfixable

redundant_closure_call: split tests into fixable

map_unit_fn: make test rustfixable

map_unit_fn: fix applicability

option_map_unit_fn: Split into fixable/unfixable

map_unit_fn: rename tests to fixable

non_copy_const: remove incorrect suggestion

needless_return: add allow()s to test, make rustfixable

needless_collect: fix suggestion, make test rustfixable

needless_borrowed_ref: fix false positive, make rustfixable

needless_borrow: allow other lints, make fixable

mem_discriminant: split test, make rustfixable

map_flatten: make it a rustfix test

Leave note on non-rustfixable tests

Remove large-digit-groups test from literals.rs

map_entry test: Fix semicolon, add run-rustfix

Split map_entry tests into fixable and unfixable

Auto merge of #4574 - Manishearth:rustup, r=yaahc,centril

Rustup to rustc 1.39.0-nightly (acf7b50c7 2019-09-25)

changelog: none

fixes https://github.com/rust-lang/rust/issues/64777

r? @phansch @yaahc

Remove suggestion for complex map_entry cases

allow osx failures

ignore single-match for or patterns

arm.pats -> arm.pat

Rustup to rustc 1.39.0-nightly (acf7b50c7 2019-09-25)

 - Addresses inference error
 - Updates compiletest

Auto merge of #4567 - phansch:toplevel_ref_arg, r=flip1995

Add run-rustfix for toplevel_ref_arg lint

changelog: none

cc #3630

Add run-rustfix for toplevel_ref_arg lint

Auto merge of #4561 - rust-lang:let-return-fix, r=phansch

Fix let_and_return lint

This fixes #4555 (false positive for the `let_and_return` lint).

changelog: none

Fix let_and_return lint

Auto merge of #4566 - phansch:update_changelog2, r=flip1995

Update the changelog for Rust 1.38

[Rendered](https://github.com/phansch/rust-clippy/blob/update_changelog2/CHANGELOG.md#rust-138)

changelog: none

Auto merge of #4537 - mikerite:unneeded_wildcard_pattern, r=phansch

Add `unneeded-wildcard-pattern` lint

changelog: Add `unneeded-wildcard-pattern` lint

Merge branch 'master' into unneeded_wildcard_pattern

Update the changelog for Rust 1.38

Auto merge of #4518 - imp:4517_license-file, r=phansch

clippy::cargo_common_metadata: check for license-file

When license property is missing in Cargo.toml check for license-file
as it may be used instead of the former. The check implemented here is
very naive as it only verifies that the field is present and is not
empty. More scrutiny can be applied by verifying the file is actually
present.

Fixes #4517

changelog: clippy::cargo_common_metadata now checks for license-file when license is missing

Auto merge of #4565 - matthiaskrgr:rustup_16, r=phansch

rustup https://github.com/rust-lang/rust/pull/64666

changelog: none

rustup https://github.com/rust-lang/rust/pull/64666

clippy::cargo_common_metadata: check for license-file

When license property is missing in Cargo.toml check for license-file
as it may be used instead of the former. The check implemented here is
very naive as it only verifies that the field is present and is not
empty. More scrutiny can be applied by verifying the file is actually
present.

Fixes #4517

Add additional tests to unneeded_wildcard_pattern

Merge branch 'master' into unneeded_wildcard_pattern

Auto merge of #4562 - phansch:wildcard_enum_match_rustfix, r=llogiq

Add run-rustfix for wildcard_enum_match_arm lint

changelog: none

cc #3630

Add run-rustfix for wildcard_enum_match_arm lint

Auto merge of #4558 - Manishearth:suggestions, r=phansch

Make more tests rustfixable

changelog: Fix various lint suggestions

Progress towards https://github.com/rust-lang/rust-clippy/issues/3630

r? @phansch

inline_fn_without_body: make it use a rustfix page

implicit_return: make it use a rustfix test

identity_conversion: make it use a rustfix test

for_loop: Split test into fixable/unfixable, make needless_range_loop use updated range syntax

eq_op: stop testing nonminimal_bool in same file

op_ref: Move tests out of eq_op file

Auto merge of #4511 - rust-lang:replace_uninitialized, r=flip1995

New lint: mem_replace_with_uninit

changelog: add `mem_replace_uninit` lint

This fixes #4485

deref_addrof_double_trigger: Note why rustfix can't be used

assign_ops: Remove MachineApplicable

new lint: mem-replace-with-uninit

Auto merge of #4544 - JoshMcguigan:issue-4542, r=flip1995

#4542 remove machine applicable suggestion

This helps #4542 (but does not completely resolve) by removing the machine applicable suggestion (which was incorrect) for that case.

I would have preferred to fix the machine applicable suggestion to handle format strings, but that's a bit beyond my current understanding of the clippy codebase. I'd be happy to give it a try given some guidance.

changelog: only produce machine applicable suggestions on `explicit_write` lint

Auto merge of #4539 - jolson88:cast-lossless-pedantic, r=flip1995

Changes cast-lossless to a pedantic lint

As discussed in #4528, this moves the cast-lossless lint from `all` to `pedantic`.

I couldn't tell from description alone if it should also be removed from the complexity category, so I left it as part of complexity for now. I didn't see any impact to the tests from this change, but I could be wrong (as this is my first PR).

fixes #4528

changelog: Moves cast-lossless from default to checking only as a `pedantic` lint.

Auto merge of #4535 - rust-lang:unsafe-doc, r=flip1995

New lint: Require `# Safety` section in pub unsafe fn docs

changelog: add `missing_safety_doc` lint

This fixes #2207

New lint: Require `# Safety` section in pub unsafe fn docs

Auto merge of #4551 - mikerite:fix-ice-reporting, r=llogiq

Fix ice reporting

changelog: none

Work around qpath_res issue

remove machine applicable suggestion explicit_write format #4542

Add note about refactoring

Co-Authored-By: Philipp Krones <hello@philkrones.com>

Add ICE regression test

Changes to catch_fatal_errors in rustc driver

A [recent PR](https://github.com/rust-lang/rust/pull/60584/files#diff-707a0eda6b2f1a0537abc3d23133748cL1151)
changed the function name from `report_ices_to_stderr_if_any` to `catch_fatal_errors`. This PR changes to using
the new function name.

Changes cast-lossless to a pedantic lint

Fixes #4528

Auto merge of #4513 - matthiaskrgr:dogdood_incrcomp, r=llogiq

tests: disable incremental compilation in dogfood to reduce target dir size

tests: execute dogfood tests with incremental compilation disabled reduces target/ dir size of when "cargo test"ing by around 2 gigs.

changelog: none

Auto merge of #4540 - jolson88:fix-place-projection, r=phansch

Fix rustc breaking change: convert to Place's new boxed slice projection

Addressed breaking changes from rust-lang PR https://github.com/rust-lang/rust/pull/63420/

I'm not entirely sure the semantics are preserved as I don't have much knowledge about MIR yet. So this code was largely reverse-engineered from the PR above. I wouldn't be surprised if I did something wrong :).

I followed the instructions to pull latest rustc from master and verified the build break I was seeing in my PR for cast-lossless in Travis CI. With these changes, it compiles again and all tests pass.

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

changelog: none

Convert to Place's new boxed slice projection

Fixes breaking change from rust-lang PR https://github.com/rust-lang/rust/pull/63420/

Apply review suggestions

Fix grammar errors and use `Pat::is_rest` instead of own function.

Auto merge of #4536 - matthiaskrgr:impl_lint_doc, r=flip1995

docs: fix broken link in adding_lints documantation

changelog: none

Auto merge of #4538 - mikerite:fix-3567, r=flip1995

Verify that issue #3567 is resolved

Closes #3567

changelog: none

Verify that issue #3567 is resolved

Closes #3567

Fix lint warnings

Add `unneeded-wildcard-pattern` lint

docs: fix broken link in adding_lints documantation

tests: dogfood: run tests with debuginfo=0 reducing dogfood target dir size even further.

tests: execute dogfood tests with incremental compilation disabled

reduces target/ dir size of when "cago test"ing by around 2 gigs.

Auto merge of #4478 - tsurai:master, r=flip1995

Fix incorrect swap suggestion

Clippy suggests using swap on fields belonging to the same owner causing two mutable borrows of the owner.

Disclosure: This is my first time working with clippy and rusts HIR. I'm very grateful for assistance and suggestions to improve the PR.

fixes #981
changelog: Fix false positive in `manual_swap` lint

Auto merge of #4534 - matthiaskrgr:fmt_, r=flip1995

run rustfmt

I was getting local `cargo test` failures because of this, yet travis was green.

changelog: none

run rustfmt

Auto merge of #4532 - rust-lang:integer-const, r=oli-obk

New `is_integer_const` to check more const ints

This mostly affects loop checks and the modulo_one lint. Tests were also updated where applicable.

changelog: none

New `is_integer_const` to check more const ints

This mostly affects loop checks and the modulo_one lint. Tests
were also updated where applicable.

Auto merge of #4525 - mikerite:use-self-constructor, r=phansch

Extend `use_self` to check constructor

Rust did not allow this before.

changelog: Extend `use_self` to check constructor

Auto merge of #4529 - lzutao:rm-feature-bind-by-move, r=matthiaskrgr

Remove feature gate bind-by-move

This feature was stabilized in https://github.com/rust-lang/rust/pull/63118
changelog: none

Remove feature gate bind-by-move

Auto merge of #4527 - rust-lang:more-vec-diag-items, r=oli-obk

Changed more `Vec` paths to diagnostic_items

In #4519, I missed a few instances of path matching for `Vec`, so here they are.

r? @oli-obk

changelog: none

Auto merge of #4522 - mikerite:fix-4514, r=phansch

Fix `or_fun_call` bad suggestion

Closes #4514

changelog: Fix `or_fun_call` bad suggestion

Changed more Vec paths to diagnostic_items

Auto merge of #4458 - flip1995:block_in_if_ext_macro, r=phansch

Allow block_in_if_{stmt,expr} in external macro

I found this by running `cargo fix --clippy` on quite a big codebase.

You could refactor this assert to
```rust
let block_expr = _;
assert!(block_expr);
```

but,
1. it doesn't increase the readability IMO
2. That isn't possible in a `debug_assert!`

I'm not sure though, if we should allow this for macros in general or just for external macros.

changelog: Allow `block_in_if_{stmt,expr}` in external macros

Auto merge of #4510 - lzutao:unsep-literals-regression-macro-attr, r=flip1995

Fix regression in case of proc-macro attribute expansion

cc  #4507
changelog: none
r? @flip1995

Add a regression test for proc-macro attribute

Fix regression in case of proc-macro attribute expansion

Extend `use_self` to check constructor

Rust did not allow this before.

Merge branch 'master' into fix-4514

Auto merge of #4519 - rust-lang:vec-diagnostic-item, r=@oli-obk

Use diagnostic item for `Vec`

This adds a new `is_type_diagnostic_item` to check items without lookup via `match_type`.

changelog: none

Use diagnostic item for