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)

Mention configuration option for too_many_lines lint

Update changelog for Rust 1.34.0

Auto merge of #3946 - rchaser53:issue-3920, r=flip1995

fix format does not parse escaped braces error

related: https://github.com/rust-lang/rust-clippy/issues/3920

fix format does not parse escaped braces  error

Auto merge of #3945 - flip1995:rustup, r=phansch

Rustup

cc https://github.com/rust-lang/rust/pull/59227#issuecomment-482411861

This fix is obsolet once rust-lang/rust#59779 and #3926 is merged.

Fix `get_def_path`

This fix is obsolet once rust-lang/rust#59779 and #3926 is merged.

Auto merge of #3938 - phansch:more_uicleanup, r=oli-obk

UI test cleanup: Extract or_fun_call and iter_nth tests

cc #2038

UI test cleanup: Extract iter_nth tests

UI test cleanup: Extract or_fun_call tests

Auto merge of #3930 - sunjay:patch-1, r=flip1995

Mention that -D warnings will deny ALL warnings, not just clippy warnings

If we pass `-D warnings` to clippy, it causes the build to fail if there are *any* warnings, even the ones not generated by clippy. This isn't immediately obvious when you're looking at this as someone just setting up clippy, so people might not expect this nor know how to fix it. I've added a few sentences in the README to help anyone who runs into this.

These docs are useful for anyone setting up clippy warnings to be denied during CI, but still otherwise want rustc warnings to be allowed.

I could have also changed `-D warnings` to `-D clippy::all` in the Travis configurations themselves, but I wasn't sure what you would prefer to have people use as the default.

Update README.md

Co-Authored-By: sunjay <sunjay@users.noreply.github.com>

Auto merge of #3931 - phansch:3891, r=flip1995

Fix ICE in decimal_literal_representation lint

Handling the integer parsing properly instead of just unwrapping.

Note that the test is not catching the ICE because plain UI tests
[currently hide ICEs][compiletest_issue]. Once that issue is fixed, this
test would fail properly again.

Fixes #3891

[compiletest_issue]: https://github.com/laumann/compiletest-rs/issues/169

Refactor check_lit method

Auto merge of #3937 - stepnivlk:type_dependent_defs-wrappers, r=oli-obk

Support updated type dependent def ID getter

This PR changes all the calls to `type_dependent_defs` (with panics via `[]`) to new method `type_dependent_def_id` with more explicit `unwrap` panic.

Rust PR: https://github.com/rust-lang/rust/pull/59216
Issue author: @oli-obk

Support updated type dependent def ID getter

Auto merge of #3933 - phansch:add_rustfix_eta, r=oli-obk

Add // run-rustfix for eta.rs test

cc #3071, #3630

Auto merge of #3932 - phansch:2910, r=flip1995

Don't emit useless_attribute lint in external macros

Fixes #2910

Add // run-rustfix for eta.rs test

cc #3071, #3630

Code formatting/cleanup

Don't emit useless_attribute lint in external macros

Exclude ice-3891.rs from rustfmt run

Because the code triggers a rustc parse error which makes rustfmt fail.

Auto merge of #3925 - phansch:3741, r=flip1995

Fix ICE in suspicious_else_formatting

Fixes #3741

Fix ICE in decimal_literal_representation lint

Handling the integer parsing properly instead of just unwrapping.

Note that the test is not catching the ICE because plain UI tests
[currently hide ICEs][compiletest_issue]. Once that issue is fixed, this
test would fail properly again.

[compiletest_issue]: https://github.com/laumann/compiletest-rs/issues/169

Remove force-host and explain no-prefer-dynamic

Mention that -D warnings will deny ALL warnings, not just clippy warnings

Auto merge of #3929 - KarboniteKream:single-char-pattern, r=oli-obk

Escape a single quote in single_char_pattern hint

This PR correctly escapes single quotes in the hint for `single_char_pattern`.
For instance, the hint for `x.split("'")` was `'''`, while it should be `'\''`.

Escape a single quote in single_char_pattern hint

Auto merge of #3921 - euclio:single-char-names-multispan, r=flip1995

use a multispan for MANY_SINGLE_CHAR_NAMES

Auto merge of #3848 - felix91gr:null_transmute, r=flip1995

Transmuting known null ptr to ref

Working on implementing #628

Auto merge of #3901 - rail-rain:issue_1670, r=flip1995

Fix `explicit_counter_loop` suggestion

#1670

This code seems to me to work, but I have two question.
* Because range expression desugared in hir, `Sugg::hir` doesn't add parenthesis to range expression.  Which function is better to check range do you think, `check_for_loop_explicit_counter` or `hir_from_snippet`?
* Do you think we need to distinguish between range expression and struct expression that creates `std::ops::Range*`?

Auto merge of #3885 - mikerite:clippy_dev_toolchain, r=phansch

Add rust-toolchain for clippy_dev

AFAIK, `clippy_dev` is supposed to be built on stable. (This is why it's not a `[bin]` inside the root `Cargo.toml`.) This adds a `rust-toolchain` file for easy switching.

cargo fmt

Auto merge of #3928 - rust-lang:tmpadd, r=Manishearth

Re-add tmp feature to compiletest

Got accidentally removed in the rustup

Re-add tmp feature to compiletest

Got accidentally removed in the rustup

Auto merge of #3927 - rust-lang:rustup, r=Manishearth

Revert compiletest hacks, use latest compiletest

The libtest changes have been reverted, see https://github.com/rust-lang/rust/pull/59766,  https://github.com/laumann/compiletest-rs/pull/174

Remove libtest from deps

Re-allow clippy::identity_conversion in compiletest

Use latest compiletest-rs

Revert tests/compile-test.rs to 61aa5c957c219abe1fb2d1b5e51db8b365b4f565

Fix ICE in suspicious_else_formatting

Merge pull request #3918 from matthiaskrgr/typos

NFC: fix typos

Merge pull request #3922 from rust-lang/run-pass

Add missing `// run-pass` annotations to ICE tests

use a multispan for MANY_SINGLE_CHAR_NAMES

Add missing `// run-pass` annotations to ICE tests

compiletest UI tests do not fail when encountering panics and ICEs
unless the `// run-pass` flag is used.

(This was forgotten in https://github.com/rust-lang/rust-clippy/pull/3743)

Remove clippy_dev as dev-dependency

Merge branch 'master' into clippy_dev_toolchain

NFC: fix typos

Merge pull request #3916 from matthiaskrgr/rustup

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

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

commit: 4122d2221ec65232bc211a266d0796a4713e0d39

Add TransmutingNull Lint
* Late Lint pass, catches:
  * One liner: 0 -> null -> transmute
  * One liner: std:null() -> transmute
  * Const (which resolves to null) -> transmute
* UI Test case for Lint
* Updated test for issue 3849, because now the lint that code generated is in Clippy.
* Expanded `const.rs` miri-based Constant Folding code, to cover
  raw pointers

Merge pull request #3905 from rust-lang/compiletest

Hacky rustup

Run rustfmt

Set level of identity_conversion FP to warn

Rustup to https://github.com/rust-lang/rust/pull/58805

rustup 41316f0449025394fdca6606d3fdb3b8f37a9872

Updated source to match with recent rustc `master` toolchain changes

Fix dogfood error of question_mark lint fix

Fix question_mark lint+test

use `span_lint_and_sugg` in `explicit_counter_loop`

Fix some test failures

Hacky rustup

Merge pull request #3902 from matthiaskrgr/rustup

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

run cargo fmt

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

Change explicit_counter_loop's message to add parentheses if necessary

Change explicit_counter_loop's message to reflect original variable name

Auto merge of #3888 - phansch:extract_mistyped_literal_suffix_tests_and_rustfix, r=flip1995

Add rustfix tests for mistyped_literal_suffix lint

This moves all `mistyped_literal_suffix` tests to their own file and
enables rustfix tests for them.

cc #3603, #2038

Based on #3887

Auto merge of #3894 - markcatley:adding_lint_documentation_fixes, r=phansch

Add implementation for the EarlyLintPass trait into the Adding Lints …

…documentation.

To follow along with the adding lint documentation you need to implement this trait in order to register the lint pass.

cargo fmt

Add rustfix tests for mistyped_literal_suffix lint

This moves all `mistyped_literal_suffix` tests to their own file and
enables rustfix tests for them.

cc #3603, #2038

Based on #3887

Move some `unreadable_literal` ui tests to correct file

The moved tests were part of `tests/ui/literals.rs` before. Now they are
placed in `tests/ui/unreadable_literal.rs`, which now also runs
`rustfix` on these 4 extra let statements.

Add implementation for the EarlyLintPass trait into the Adding Lints documentation.

To follow along with the adding lint documentation you need to implement this trait in order to register the lint pass.

Auto merge of #3893 - mati865:rustup, r=oli-obk

Rustup

Supersedes https://github.com/rust-lang/rust-clippy/pull/3889

Addresses some review comments from previous PR and rustups to https://github.com/rust-lang/rust/pull/58899

Cargo fmt

Drop range_contains feature

Fix rustfmt::skip detection

Rework clippy detection in attribute lint

name -> check_name

span -> span()

name -> ident_str

NestedMetaItemKind -> NestedMetaItem

Drop redundant &

Fix path_qualified

As suggested by eddyb

Replace most of ty:Ty with Ty

Feed dog

Adapt paths to new formatting

cargo fmt

Use LocalInternedString inside of AbsolutePathPrinter

AbsolutePathBuffer -> AbsolutePathPrinter

Remove ty::TyKind from eta_reduction and replace it with ty::Ty

parent_def_id -> parent

item_path_str -> def_path_str

Reimplement AbsolutePathBuffer

Auto merge of #3886 - phansch:rustfix_useless_asref, r=flip1995

Enable rustfix for `useless_asref` lint tests

cc #3630

Enable rustfix for `useless_asref` lint tests

Auto merge of #3883 - daxpedda:missing_docs_in_private_items, r=phansch

Add `doc(include = ...)` detection to `missing_docs_in_private_items`

The whole `missing documentation in crate` part doesn't have any tests. If I should add test cases tell me.

Add rust-toolchain for clippy_dev

Another test for missing crate documentation.