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

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