Make `redundant_field_name` not care range expressions

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

Typo

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

Merge pull request #2502 from flip1995/compiletest_version

Use compiletest version v0.3.7

Use compiletest version v0.3.7

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

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)

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

Merge pull request #2448 from rust-lang-nursery/fixes

Fixes

Fixes #2426 (if_same_then_else false positive)

Fix  #2427

Remove rarely used `type_size` helper function

Merge pull request #2446 from goodmanjonathan/fix-copies-test

Make several tests actually test the correct lints

Merge pull request #2445 from guido4000/patch-1

Simplify recommended command

make the copies.rs test actually test the correct lints

Change command to run clippy without installation

Simplify recommended command

Merge pull request #2444 from phansch/fix_incorrect_useless_attribute_suggestion

Partly fix incorrect useless_attribute suggestion

Partly fix incorrect useless_attribute suggestion

This fixes an incorrect suggestion from the `useless_attribute` lint
when using `cfg_attr`.

Additionally, it will not show a suggestion anymore, if the attribute
begins on a previous line, because it is much harder to construct the
span of multi-line `cfg_attr` attributes as they don't appear in the AST.

To fix it completely, one would have to parse upwards into the file,
and find the beginning of the `cfg_attr` attribute.

Merge pull request #2443 from goodmanjonathan/fix-2438

Don't suggest eliding 'static on associated consts

don't suggest eliding 'static on associated consts

Merge pull request #2441 from flip1995/literal_rep

Make decimal_literal_representation a restriction lint

Make decimal_literal_representation a restriction lint

Merge pull request #2439 from gnieto/fix/cterror

Fix ICE comparing `ExprRange` equality

Fix ICE comparing `ExprRange`

`eq_expr` on hir::utils was throwing an ICE due to an invalid
LateContext being used. Due to this missusage, it was generating an ICE
with the code on the following issue:
https://github.com/rust-lang-nursery/rust-clippy/issues/2423

Cleanup calls to `layout_of`

Version Bump

Merge pull request #2433 from kimsnj/matches_sugg

Fix suggestions for ref matches

Merge pull request #2415 from HMPerson1/fix-2356

Fix `ImplItem`s being ignored

Merge pull request #2428 from phansch/fix_lint_list_issues

Allow empty lines in lint doc examples

Merge pull request #2419 from phansch/fix_doc_comment_false_positive

Fix false positive in empty_line_after_outer_attr

Merge pull request #2436 from messense/rustup

Rustup to rustc 1.25.0-nightly (0c6091fbd 2018-02-04)

Rustup to rustc 1.25.0-nightly (0c6091fbd 2018-02-04)

Removing extra newline

Fix suggestions for ref matches

Bump to 0.0.185

Fix `const_static_lifetime`

Merge pull request #2430 from Frederick888/fix-2429

set CodeSuggestion approximate to false. fixes #2429

Fix `non_expressive_names`

Fix `get_enclosing_block`

set CodeSuggestion approximate to false. fixes #2429

Allow empty lines in lint doc examples

This makes sure that empty lines in lint examples are preserved.

It also fixes the documentation for the invalid_ref lint, which was not
shown because of an extra newline before the lint declaration.

Give travis a guaranteed existing directory for ui test output

Merge pull request #2417 from mathstuf/gitignore-support-symlinks

gitignore: support ignoring target as symlinks

Fix false positive in empty_line_after_outer_attr

Doc comments are syntactic sugar for #[doc] attributes, so this lint was
catching them, too.

This commit makes it so that doc comments are ignored in this lint.

I think, for normal attributes it makes sense to warn about following empty
lines, for doc comments, less. This way the user has some freedom over
the formatting.

gitignore: support ignoring target as symlinks

Merge pull request #2410 from gnieto/task/questionMarkIso

Question mark lint

Update UI tests

Add question mark operator

Merge pull request #2413 from flip1995/assign_ops

Improved suggestion on misrefactored_assign_op lint

Additionally suggest the semantic equal variant

Version bump

Improved suggestion on misrefactored_assign_op lint. Fixes #1239

Merge pull request #2340 from phansch/newline_after_attributes

Warn on empty lines after outer attributes

Merge pull request #2412 from topecongiro/double-comparison

Add double comparison lint

Add double comparions lint

Rustup to rustc 1.25.0-nightly (90eb44a58 2018-01-29)

Merge pull request #2409 from montrivo/while_let_on_iterator

#1121: already fixed, adding a test

#1121: already fixed, adding a test

Add workaround for hidden outer attribute

If the snippet is empty, it's an attribute that was inserted during macro
expansion and we want to ignore those, because they could come from external
sources that the user has no control over.
For some reason these attributes don't have any expansion info on them, so
we have to check it this way until there is a better way.

Make lint work on all members of ast::Item_

Add test case for comments between item and attr

Warn on empty lines after outer attributes

Bump to 0.0.183

Bump compiletest version

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

Rustup to rustc 1.25.0-nightly (21882aad7 2018-01-28)

Update spans

mut_mut_macro is missing plugin(clippy)

Remove an unused binary file

Rustup to rustc 1.25.0-nightly (7d6e5b9da 2018-01-27)

Merge pull request #2403 from mikerite/fix_compile_2

Fix compilation

Fix compilation

Fix the compilation broken by these two changes:
+ https://github.com/rust-lang/rust/commit/2d56abfbebdc905dafc9cf9edc0a6f58e4de7cbd#diff-7fceb7ede15b205bf5ad812c31d75384L1459
+ https://github.com/rust-lang/rust/commit/ccf0d8399e1ef3ed6bf7005650ce42aa646b5cc7#diff-64b696b0ef6ad44140e973801ed82b25L2771