Undo a temporary fix for a cargo bug

Fixes #2566

Merge pull request #2565 from mikerite/issue_2420_tests_pr

Add tests to ensure that issue #2420 is resolved

Add tests to ensure that issue #2420 is resolved

The issue was probably fixed by ff32d5f7. Closes #2420.

Merge pull request #2561 from mikerite/walk_ptrs_ty_depth_pr

Minor refactoring (walk_ptrs_ty_depth)

Minor refactoring (walk_ptrs_ty_depth)

Replace `walk_ptrs_ty_depth` with `walk_ptrs_ty` when the depth value is
ignored.

Merge pull request #2539 from Baelyk/master

Add suggestion to useless_format

Add suggestion to useless_format

Resolves #2505

Suggests that you use `"foo".to_string()` instead of `format!("foo")`.

Merge remote-tracking branch 'origin/master'

Version bump

undo accidental push to master

Use rustc from latest merged PR instead of nightly

Rustup

Merge pull request #2545 from flip1995/sus_arith

Don't lint comparison operators in arithmetic impls

Don't lint comparison operators in arithmetic impls

Merge pull request #2544 from flip1995/panic

Check if the panic message was created by the assert-macro

Check if the panic message was created by the assert-macro

Merge pull request #2533 from mati865/dogfood

Code cleanup

Allow float_cmp in consts lint code

Signed-off-by: Mateusz Mikuła <mati865@gmail.com>

Remove attributes for stable features

Signed-off-by: Mateusz Mikuła <mati865@gmail.com>

Fix BiRem for floats

Signed-off-by: Mateusz Mikuła <mati865@gmail.com>

Format code

Signed-off-by: Mateusz Mikuła <mati865@gmail.com>

Apply clippy suggestions

Signed-off-by: Mateusz Mikuła <mati865@gmail.com>

Rustup

Merge pull request #2531 from mati865/master

Rustup

Fix warning

Signed-off-by: Mateusz Mikuła <mati865@gmail.com>

Update tests after merge

Signed-off-by: Mateusz Mikuła <mati865@gmail.com>

Merge branch 'miri'

Update tests

Signed-off-by: Mateusz Mikuła <mati865@gmail.com>

Rustup

Signed-off-by: Mateusz Mikuła <mati865@gmail.com>

Merge pull request #2524 from rust-lang-nursery/update_dependencies

Update dependencies

Update dependencies

Update to rustc master

Merge pull request #2520 from ordovicia/remove-duct

Remove duct dependency

Merge pull request #2519 from ordovicia/issue-2516

Fix #2516

Remove duct dependency

Fix comment

Ignore all macros in redundant_field_names

Merge pull request #2515 from kimsnj/infinite_loop

lint: while immutable condition: do not lint constants

lint: while immutable condition: do not lint constants

Merge pull request #2483 from kimsnj/infinite_loop

immutable while condition

lint: immutable condition: add internally mutable test

Merge pull request #2511 from flip1995/sus_impl

UnNeg and UnNot count as additional operations now

Merge pull request #2513 from niklasf/increase-unreadable-literal

Increase unreadable_literal digits (fixes #1958)

Increase unreadable_literal digits (fixes #1958)

UnNeg and UnNot count as additional operations now

lint: while immutable condition: refactor to use ExprUseVisitor

Merge pull request #2509 from ordovicia/redundant_field_names_range

Remove unused variable and a minor refactoring

Move call of `is_range_expression()` outside of blocks

Merge pull request #2507 from ordovicia/redundant_field_names_range

Don't lint range syntax with var name `start` and/or `end`

Unused variable is left

Merge pull request #2493 from bootandy/fix_cow

Lint passing Cow by reference

Make `redundant_field_name` not care range expressions

Hand-written `Range` struct family are treated normally.

Merge pull request #2497 from waywardmonkeys/single-char-pattern

Fix single_char_pattern for \n, \t, etc.

Merge branch 'master' into fix_cow

Merge pull request #2504 from flip1995/lit_float_repr

Fix unreadable_literal lint for scientific float notation

Typo

Merge pull request #2506 from HMPerson1/fix_2496

Fix #2496

Don't lint range syntax with var name `start` and `end`

Fix #2496

Fix unreadable_literal lint for scientific float notation

Merge pull request #2502 from flip1995/compiletest_version

Use compiletest version v0.3.7

Use compiletest version v0.3.7

Change recomendation to: &[type] from Cow<type>

Fix #2494 add suggestion for unreadable_literal

Add `rustc --explain E0308` line to relevant tests

Fix single_char_pattern for \n, \t, etc.

Single characters that are escaped weren't being searched / replaced
correctly in the hint string, so it was saying to replace, say,
`"\n"` with `"\n"` rather than `'\n'`.

lint: while loop: detect if no var from the condition is mutated

lint: while immutable condition: refactor to use hir::Visitor

Merge pull request #2495 from alexheretic/fix-2494

Fix #2494 add suggestion for unreadable_literal

Fix #2494 add suggestion for unreadable_literal

Add `rustc --explain E0308` line to relevant tests

Lint passing Cow by reference

Add lint for reference to Cow to the same place in the code where
lint for reference to String lives.

https://github.com/rust-lang-nursery/rust-clippy/issues/2405

Merge pull request #2486 from ordovicia/fix

Fix for rustc 1.26.0-nightly (322d7f7b9 2018-02-25)

Bump to 0.0.187

oops

Merge pull request #2489 from rust-lang-nursery/rustup

Rustup to rustc 1.26.0-nightly (322d7f7b9 2018-02-25)

Rustup to rustc 1.26.0-nightly (322d7f7b9 2018-02-25)

Fix for rustc 1.26.0-nightly (322d7f7b9 2018-02-25)

lint: immutable only vars in while condition

Merge pull request #2479 from gnieto/fix/authorLint

Fix author lint

Remove explicit visit_qpath method

Instead of replacing the default behaviour of the visit_qpath method,
I've moved the printing code to private method of PrintVisitor
(print_qpath).

Fix author lint

The author lint was generating invalid code as shown on issue:
https://github.com/rust-lang-nursery/rust-clippy/issues/2442

I've changed the generated code to properly track cast
expressions.

Unfortunatelly, I've had to rewrite the `visit_decl` method, to
avoid that last if of the chain will be added. After looking at the code,
this last line was being added because of the `let x: char` part, but not
because of the `0x45df as char` expression.

It seems that let statements should not generate code on the author
lint, but I'm not sure that this is true or if I'm breaking
something on other code generation parts.

Finally, I've added a test for the author lint, but I'm not sure that
this needs to be added to the testsuite.

Merge branch 'rustup'

Update Rust to 063deba92e

Merge pull request #2471 from bootandy/fix_span2

Fix: point to correct problem part of code, update test

Do not pass Span by reference

Merge pull request #2473 from phansch/handle_multiline_attributes

Lint multiline attributes properly

Lint multiline attributes properly

This makes it so that the `empty_line_after_outer_attribute` lint only
checks for newlines between the end of the attribute and the beginning
of the following item.

We need to check for the empty line count being bigger than 2 because
now the snippet of valid code contains only `\n` and splitting it
produces `["", ""]`
Invalid code will contain more than 2 empty strings.

Stop unwanted newlines being applied on unwrap_or

Simplify creation of span_replace_word

Part of unwrap_or test

Add newlines in unwrap_or ui test

Merge pull request #2459 from phansch/mention_how_to_run_single_test

Explain how to execute a single UI test

Merge pull request #2458 from flip1995/suspicious_impl

Lint for suspicious implementations of arithmetic std::ops traits

Merge pull request #2470 from flip1995/doc_conf

Fix name of configuration parameters in documentation

Fix: point to correct problem part of code

Fix span so it no longer contains the whole train-wreck of code and only
points to the problem function (for the unwrap_or lint).

https://github.com/rust-lang-nursery/rust-clippy/issues/2422

Update ui test methods - it had several cases where the error message
span is now shorter

Fix name of configuration parameters in documentation

Merge branch 'master' into suspicious_impl

Merge pull request #2451 from TomasKralCZ/rdn-fields

Implement redundant field names lint #2244

Explain how to execute a single UI test

Lint for suspicious implementations of arithmetic std::ops traits

Suggestion fixed, simplified lint logic.

Implement redundant field names lint #2244