Auto merge of #4030 - phansch:rustup190425, r=Manishearth

Rustup for https://github.com/rust-lang/rust/pull/59042

changelog: none

Rustup for https://github.com/rust-lang/rust/pull/59042

Auto merge of #4027 - kraai:remove-approx_constant-known-problems, r=phansch

Remove approx_constant known problems

Fixes #4025.

changelog: Remove `approx_constant`'s documentation's "Known problems" section.

Auto merge of #4023 - phansch:add_test, r=flip1995

Add test for or_fun_call macro suggestion

changelog: none

Closes #1018

Remove approx_constant known problems

Fixes #4025.

Auto merge of #4022 - phansch:move_some_loop_tests, r=flip1995

UI test cleanup: Extract needless_range_loop tests

changelog: none

Auto merge of #4024 - kraai:suppress-let_and_return-if-let-has-attributes, r=flip1995

Suppress let_and_return if let has attributes

Fixes #3882.

changelog: suppress `let_and_return` if `let` has attributes

Suppress let_and_return if let has attributes

Fixes #3882.

Add test for or_fun_call macro suggestion

Closes #1018

UI test cleanup: Extract needless_range_loop tests

Auto merge of #4021 - kraai:change-cast_lossless-warning, r=Manishearth

Change "if types change" to "if you later change the type"

Fixes #3964.

changelog: change wording of `cast_lossless` message

Change "if types change" to "if you later change the type"

Fixes #3964.

Auto merge of #4018 - rust-lang:or_fn_call_variants, r=oli-obk

Ignore non-const ctor expressions in or_fn_call

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

Should have been fixed by #919, however that focuses on const ctor expressions only, and `.or(Some(local))` isn't const.

This also automatically ignores things like `.or(Some(local.clone())` which we don't actually want to do; I need to figure out what to do here.

changelog: Fixed false positive in [`or_fn_call`] pertaining to enum variant constructors

r? @oli-obk @phansch

Auto merge of #4020 - rust-lang:rustup, r=phansch

Rustup to rustc 1.36.0-nightly (fe0a415b4 2019-04-23)

r? @oli-obk @phansch

Rustup to rustc 1.36.0-nightly (fe0a415b4 2019-04-23)

Add test

Ignore all enum and struct constructors in lints about `*or(call())`

Auto merge of #4017 - Xanewok:wip-disambiguate-extern, r=oli-obk

compiletest: Disambiguate extern crate deps shared with the compiler

Fixes #4015.

changelog: Handle deps shared with the compiler in the internal compiletest suite

Attempts to fix the multiple matching crates error using the `--extern dep=path` disambiguation. This only includes `serde` at the moment because it's the only problematic dep right now (is inside Rust sysroot and pulled via `extern crate` in the test suite).

I'm not exactly sure this is the right approach (FWIW it fixes the issue locally), please do tell if this should be done differently.

compiletest: Disambiguate extern crate deps shared with the compiler

Auto merge of #4013 - kraai:move-path_buf_push_overwrite-to-nursery, r=matthiaskrgr

Move path_buf_push_overwrite to nursery

See #4012.

changelog: move `path_buf_push_overwrite` to the nursery

Auto merge of #4014 - Xanewok:rustup-60124, r=matthiaskrgr

rustup: Use newly-introduced mutability query for statics

Fixes fallout from https://github.com/rust-lang/rust/pull/60124.
cc https://github.com/rust-lang/rust/issues/60154

changelog: none

Use newly-introduced mutability query for statics

Fixes fallout from https://github.com/rust-lang/rust/pull/60124.
cc https://github.com/rust-lang/rust/issues/60154

Move path_buf_push_overwrite to nursery

See #4012.

Auto merge of #4011 - phansch:add_test_for_derive_expansion, r=flip1995

Add test for derives for used_underscore_binding lint

This closes #852 as I can't reproduce the original issue anymore.

changelog: none

Auto merge of #4007 - phansch:fix_allowing_toplevel_ref_arg, r=flip1995

Allow allowing of toplevel_ref_arg lint

I'm not sure why some lints need the `HirId` to be able to recognize the
lint level attributes, but this commit makes the lint level attributes
work for `toplevel_ref_arg`.

Fixes #2332

changelog: Allow allowing of `toplevel_ref_arg` lint

Auto merge of #4005 - phansch:rustfix_match_as_ref, r=flip1995

Add run-rustfix for match_as_ref lint

* Extracts `match_as_ref` into separate file
* Adds `// run-rustfix` to `tests/ui/match_as_ref.rs`

cc #3630

changelog: none

Auto merge of #4006 - phansch:fix_module_name_repetitions_fp, r=flip1995

Fix false positive in module_name_repetitions lint

This lint was triggering on modules inside expanded attrs, like
for example `#[cfg(test)]` and possibly more.

It was not reporting a location in #3892 because `span.lo()` and `span.hi()` both were 0.

Fixes #3892

changelog: Fix false positive in `module_name_repetitions` lint

Add test for derives for used_underscore_binding lint

This closes #852 as I can't reproduce the original issue anymore.

Auto merge of #4009 - phansch:update_compiletest, r=phansch

Update compiletest_rs

This includes a fix that make ICEs appear in UI tests again.

Update compiletest_rs

This includes a fix that make ICEs appear in UI tests again.

Allow allowing of toplevel_ref_arg lint

I'm not sure why some lints need the `HirId` to be able to recognize the
lint level attributes, but this commit makes the lint level attributes
work for `toplevel_ref_arg`.

Fix false positive in module_name_repetitions lint

This lint was triggering on modules inside expanded attrs, like
for example `#[cfg(test)]` and possibly more.

Add run-rustfix for match_as_ref lint

* Extracts `match_as_ref` into separate file
* Adds `// run-rustfix` to `tests/ui/match_as_ref.rs`

Auto merge of #3989 - flip1995:assert_on_const, r=phansch

Don't trigger assertions_on_constants on debug_assert!(false)

Fixes #3948
Fixes #3765

changelog: Fix `debug_assert!` false positive on `assertions_on_constants` lint

Auto merge of #3990 - felixrabe:patch-1, r=flip1995

Typo

changelog: none

Auto merge of #3978 - phansch:rustfix_len_zero, r=flip1995

Add run-rustfix for len_zero lint

* Extracts len_without_is_empty into separate file
* Adds `// run-rustfix` to `tests/ui/len_zero.rs`

cc #3630

Typo

Fix dogfood error

Auto merge of #3954 - andrehjr:add-lint-path-buf-overwrite, r=flip1995

Add Lint PathBufPushOverwrite

Closes #3923

This is a very simple Lint that checks if push is being called with a Root Path. Because that can make it overwrite the previous path.

I used std::path::Path to check if it's root, in order to keep it working across windows/linux environments instead of checking for '/'. Is that alright?

On the `if_chain!` block, Is there a way to make it short while getting the value of the first argument? I got the example from other lints.

Note that this is first Lint, I hope I got everything covered  🚀

Add lint PathBufPushOverwrite

Auto merge of #3985 - phansch:move_some_cast_tests, r=flip1995

Move two cast_lossless tests to their correct files

First part of checking off the `tests/ui/cast.rs` checkbox in #3630.

Update *.stderr file

Remove code duplication

Format code

Add test for debug_assert!(false)

Don't lint debug_assert!(false)

Auto merge of #3987 - phansch:rustfix_option_map_or_none, r=flip1995

Add run-rustfix for option_map_or_none lint

* Extracts `option_map_or_none` tests into separate file
 * Adds `// run-rustfix` to `tests/ui/option_map_or_none.rs`

cc #3630

Auto merge of #3984 - phansch:bytecount_sugg, r=flip1995

Change naive_bytecount applicability to MaybeIncorrect

We can't use `MachineApplicable` here as applying the fix will cause
another error because `bytecount` would first have to be added to the
Cargo.toml.

Example:

```
error: You appear to be counting bytes the naive way
  --> $DIR/bytecount.rs:5:13
   |
LL |     let _ = x.iter().filter(|&&a| a == 0).count(); // naive byte count
   |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: Consider using the bytecount crate: `bytecount::count(x, 0)`
```

Just replacing it with the suggestion is not enough.

cc #3630

Auto merge of #3968 - kraai:lint-pass-macros, r=flip1995

Lint pass macros

Add run-rustfix for option_map_or_none lint

Move two cast_lossless tests to their correct files

First part of checking off the `tests/ui/cast.rs` checkbox in #3630.

Change naive_bytecount applicability MaybeIncorrect

We can't use `MachineApplicable` here as applying the fix will cause
another error because `bytecount` would first have to be added to the
Cargo.toml.

Add tests for declare_lint_pass and impl_lint_pass

Use lint pass macros

Fixes #3917.

Document `declare_lint_pass!`

Fix lint_without_lint_pass internal lint

Auto merge of #3926 - flip1995:def_path_uplift, r=phansch

Use {get,match}_def_path from rustc

This is a follow up of the uplift of `{get,match}_def_path` to rustc.

~~This is blocked on rust-lang/rust#59779~~

Use {get,match}_def_path from LateContext

Remove uplifted functions {get,match}_def_path from Clippy

Auto merge of #3966 - flip1995:internal_lints, r=oli-obk

Enable rustc internal lints

Closes #3965

I'm not 100% sure if enabling the `-Zunstable-options` flag in the `.cargo/config` file is the right place.

Auto merge of #3977 - phansch:add_rustfix_bool_comparison, r=flip1995

Add run-rustfix for bool_comparison lint

cc #3630

Auto merge of #3976 - phansch:deref_addrof_rustfix, r=flip1995

Add run-rustfix for deref_addrof lint

* renames `tests/ui/reference.{rs,stderr}` to
  `tests/ui/deref_addrof.{rs,stderr}
* Moves small part of the testfile to a separate file as the lint
  triggered again on the fixed code (as intended)
* Adds `// run-rustfix` to `tests/ui/deref_addrof.rs`

cc #3630

Auto merge of #3975 - phansch:has_placeholders, r=flip1995

Change while_let_loop applicability to HasPlaceholders

The suggestion has been changed at some point to use `..` in the suggested code.
Due to that we can't make the lint MachineApplicable anymore.

cc #3630

Auto merge of #3972 - rust-lang:oli-obk-patch-1, r=flip1995,Manishearth

Create PULL_REQUEST_TEMPLATE

changelog: none

addresses https://github.com/rust-lang/rust-clippy/issues/3955#issuecomment-483567400

Add run-rustfix for len_zero lint

* Extracts len_without_is_empty into separate file
* Adds `// run-rustfix` to `tests/ui/len_zero.rs`

Add run-rustfix for bool_comparison lint

Add run-rustfix for deref_addrof lint

* renames `tests/ui/reference.{rs,stderr}` to
  `tests/ui/deref_addrof.{rs,stderr}
* Moves small part of the testfile to a separate file as the lint
  triggered again on the fixed code (as intended)
* Adds `// run-rustfix` to `tests/ui/deref_addrof.rs`

while_let_loop uses placeholders in suggestions

Due to that we can't make the lint MachineApplicable anymore.

Remove rust-toolchain file from clippy_dev

Update adding_lints.md

Update PULL_REQUEST_TEMPLATE

Add new lint checklist

Create PULL_REQUEST_TEMPLATE

Auto merge of #3970 - rust-lang:map_copied, r=flip1995

Suggest .copied() instead of .cloned() in map_clone where applicable

partial fix for https://github.com/rust-lang/rust-clippy/issues/3958

changelog: Improve suggestion in `map_clone` to suggest `.copied()` where applicable

Only suggest .copied() for Option right now

Also suggest .copied() when .clone() is called on a Copy type

Suggest .copied() instead of .cloned() in map_clone when dealing with references

Deny rustc internal lints

Remove clippy::default_hash_types internal lint

Enable -Zunstable-options in .cargo/config

Auto merge of #3961 - rust-lang:rustup, r=matthiaskrgr

Rust upgrade to rustc 1.35.0-nightly (9cd61f025 2019-04-14)

Handles breakages from https://github.com/rust-lang/rust/pull/59877

r? @oli-obk @matthiaskrgr

Clean up unused cx parameters

HirIdify ReadVisitor

Use _from_hir_id APIs

Remove now-unnecessary calls to node_to_hir_id

Auto merge of #3960 - phansch:fix_except, r=flip1995

Remove `except` in suspicious_else_formatting

96c34e85 contains the fix:

This was causing two different ICEs in #3741. The first was fixed in #3925.

The second one is fixed with this commit: We just don't `expect` anymore.
If the snippet doesn't contain an `else`, we stop emitting the lint because
it's not a suspiciously formatted else anyway.

Unfortunately I wasn't able to provide a minimal test case, but I think it's
fine since it's just ignoring the `None` case now.

And ad27e3ff cleans up the lint code to use `if_chain`.

Fixes #3741 once more.

Auto merge of #3956 - rust-lang:beta-clog, r=phansch

Beta changelog

Maintain a changelog for beta so the release team can easily include important additions in the release blog posts.

r? @phansch, @oli-obk

Link to ICEs

Update CHANGELOG.md

Co-Authored-By: Manishearth <manishsmail@gmail.com>

cargo fmt

Auto merge of #3951 - phansch:add_combine_integration_test, r=phansch

Add Marwes/combine to integration tests

link: https://github.com/Marwes/combine
`combine` uses a lot of macros internally, has been downloaded more than 200_000
times and is also a dependency of [`redis-rs`](https://crates.io/crates/redis).

Clippy also previously ICEd on combine in #3747 so I think it would be
good to have this crate as an integration test.

Refactor suspicious_else_formatting using if_chain

Remove `except` in suspicious_else_formatting

This was causing two different ICEs in #3741.
The first was fixed in #3925.

The second one is fixed with this commit: We just don't `expect`
anymore. If the snippet doesn't contain an `else`, we stop emitting the
lint because it's not a suspiciously formatted else anyway.

Auto merge of #3959 - rust-lang:rustup, r=phansch

Rust upgrade to rustc 1.35.0-nightly (00856722b 2019-04-13)

r? @phansch @oli-obk

Rust upgrade to rustc 1.35.0-nightly (00856722b 2019-04-13)

Beta changelog

Auto merge of #3950 - phansch:update_changelog, r=Manishearth

Update changelog for Rust 1.34.0

[Rendered](https://github.com/phansch/rust-clippy/blob/update_changelog/CHANGELOG.md#rust-134-2019-04-10)

Add Marwes/combine to integration tests

link: https://github.com/Marwes/combine
`combine` uses a lot of macros internally, has been downloaded more than 200_000
times and is also a dependency of [`redis-rs`](https://crates.io/crates/redis).

Clippy also previously ICEd on combine in #3747 so I think it would be
good to have this crate as an integration test.

Mention configuration option for too_many_lines lint

Update changelog for Rust 1.34.0