Merge pull request #2644 from phansch/fix_nonminimal_bool_false_positive

Fix nonminimal_bool false positive

Fix nonminimal_bool false positive

It was checking any is_ok, is_err, is_some, is_none method for negation
but it should only perform the check for the built-in types, not custom
types.

Merge pull request #2643 from phansch/split_up_match_bool_ui_tests

Split up match_bool UI tests

Split up match_bool UI tests

Merge pull request #2641 from rcoh/patch-2

Update hash_xor_eq docs to match behavior

Merge pull request #2640 from mikerite/fix_compilation_20180406

Fix compilation for nightly 2018-04-06

Fix docs to match behavior

Fix compilation for nightly 2018-04-06

Breakages for introduced by rust pull request
'AST: Give spans to all identifies' - rust-lang/rust/pull#49154

Closes #2639

Merge pull request #2633 from mikerite/ref_cow_tests

Move ref cow tests

Merge pull request #2635 from phansch/split_up_some_match_ui_tests

Split up some single_match UI tests

Split up some single_match UI tests

This moves only the single_match tests over to the new file.

Move ref cow tests

This commit moves the ref cow tests from needless_borrow.rs to ptr_arg.rs
where all the other PTR_ARG tests are.

Merge pull request #2632 from phansch/fix_useless_format_false_positive

Fix useless_format false positive with macros

Fix useless_format false positive with macros

Clippy was issuing a warning when `format!` was used inside a macro.
That's a problem because macros have different syntax and can be outside
the control of the user.

This skips the `useless_format` check if the `format!` call is inside a
macro.

argh

Merge pull request #2629 from memoryleak47/typo

typo

typo

Fix driver dogfood bug

Bump version to 0.0.192

Update changelog

Remove uses of ExprKind::Inplace

Merge pull request #2628 from TimNN/patch-2

Add missing `clippy_` prefix to lint groups in Readme

Add missing `clippy_` prefix to lint groups in Readme

Merge pull request #2625 from mikerite/clippy_warning

Fix clippy warning

fix other instance of internals docs

Fix clippy warning

new internals docs

Merge pull request #2622 from flip1995/imm_while_fields

Don't trigger while_immutable_condition for mutable fields

Don't trigger while_immutable_condition for mutable fields of tuples/structs

Merge pull request #2616 from mikerite/reenable_win_dogfood

Re-enable dogfood test on Windows

Merge pull request #2618 from rcoh/patch-1

Move `set -e` to before the deploy

Move `set -e` to before the deploy

I _think_ this might be why the deploy script crashing isn't causing the release to fail (see #2600)

Re-enable dogfood test on Windows

This should work now that dogfood uses a separate output directory.

Merge pull request #2608 from mrecachinas/feature/print-string-literal

Check for literals as println! args

Revert adding env! to tests

Add edge case with env! arg to test and known problems

Merge pull request #2612 from phansch/document_how_to_use_the_author_lint

Document the author lint

Document the author lint

Merge pull request #2611 from phansch/improve_readme

Readme: Explain nightly install and clippy update

Merge pull request #2605 from mikerite/dogfood_target_dir

Make dogfood test output to seperate directory

Readme: Explain nightly install and clippy update

Support non-unicode paths for dogfood test

Fix clippy warning

Merge branch 'master' into dogfood_target_dir

Fix clippy warning

Fix too_many_arguments on `check_general_case` by allowing it. I can't
see a sensible way of grouping the parameters.

Fix clippy warning

Fix cyclomatic_complexity warning on `check_expr` by allowing it. This
is preferable to increasing the threshold every time the method changes.

Fix clippy warning

Fix `option_option` warning on `to_const_range` by taking the entire range
as an parameter instead of the start and end.

Fix clippy warning

Allow `many_single_char_names` on `SpanlessHash::hash_expr`. Each
variable has a small scope and the method is readable.

Change declare_lint! to declare_clippy_lint!

Add #![allow(print_literal)] to other test/ui/print_*.rs tests

Add implementation and tests for literal checking in print/println format args

Merge pull request #2606 from phansch/improve_contributing_instructions

Improve CONTRIBUTING.md

Improve CONTRIBUTING.md

* Incremental compilation is on by default
* Restructured the label overview to go from easy to more difficult labels.

Fix clippy warnings from last commit

Make dogfood test output to seperate directory

This commit makes `cargo clippy` output the build artifacts to a
separate directory if the `CLIPPY_DOGFOOD` env var is set. This should
prevent dogfood builds from interfering with regular builds.

This should help with issue #2595.

Merge pull request #2601 from flip1995/export

Fix lintlib script

Fix lintlib script

Merge pull request #2590 from phansch/fix_another_false_positive

Fix false positive in empty_line_after_outer_attr

Handle nested block comments

Move empty_line_after_outer_attribute to nursery

From the clippy side it's difficult to detect empty lines between
an attributes and the following item because empty lines and comments
are not part of the AST. The parsing currently works for basic cases
but is not perfect and can cause false positives.

Maybe libsyntax 2.0 will fix some of the problems around attributes but
comments will probably be never part of the AST so we would still have
to do some manual parsing.

Fix false positive in empty_line_after_outer_attr

Before, when you had a block comment between an attribute and the
following item like this:

```rust
\#[crate_type = "lib"]
/*

*/
pub struct Rust;
```

It would cause a false positive on the lint, because there is an empty
line inside the block comment.

This makes sure that basic block comments are detected and removed from
the snippet that was created before.

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

Rustup to rustc 1.26.0-nightly (ae544ee1c 2018-03-29)

Bump to 0.191

Update changelog for 0.191

Rustup to rustc 1.26.0-nightly (ae544ee1c 2018-03-29)

Merge pull request #2589 from rust-lang-nursery/rangearg

Move RangeArgument

Merge pull request #2579 from rust-lang-nursery/lint_audit_mcve

lint audit: Implementation + move one lint

Address review comments

Move RangeArgument

Document lint groups

Merge pull request #2585 from bjgill/use-check-manifest-handling

Use cargo check's manifest handling

Delete all code for handling manifest path

Now that we're using cargo check, we can stop needing to find out the
manifest path ourselves. Instead, we can delegate to cargo check, which
is perfectly capable of working out for itself what needs to be built.

This fixes #1707 and #2518.

Note that this PR will change the output. We will no longer output `bin:
foo` before each crate. This a bit unfortunate. However, given that
we're now going to be building in parallel (which is *much* faster), I
think this is acceptable - we'll be no worse than cargo itself.

Update test suite

Categorize all the lints!

Merge pull request #2582 from rust-lang-nursery/clippy_aint_no_compiler

Use cargo check instead of cargo rustc

Use cargo check instead of cargo rustc

Implementation + move one lint

Undo current_dir -> CARGO_MANIFEST_DIR move

Version bump

Merge pull request #2572 from flip1995/immut_while

Fix check of immutable condition in closure

Merge pull request #2578 from mikerite/fix_issue_2397_pr_2

Fix enum_glob_use false positives

Fix enum_glob_use false positives

Closes #2397.

This checks the def of the `ItemUse` path instead of checking the
capitalization of the path segements. It was noted that this def would
sometimes be `Def::Mod` instead of `Def::Enum` but it seems correct now.

Skip the mutation in while body case for closures

Fix check of immutable condition in closure

Merge pull request #2576 from kimsnj/infinite_loop

while_immutable_condition: fix handling of self

while_immutable_condition: limit suggestion span to condition

while_immutable_condition: fix handling of self

Merge pull request #2577 from phansch/false_positive_empty_line_after_outer_attr_with_enum

Fix false positive in empty_line_after_outer_attribute

Fix false positive in empty_line_after_outer_attribute

`empty_line_after_outer_attribute` produced a false positive warning when
deriving `Copy` and/or `Clone` for an item.

It looks like the second point in [this comment][that_comment] is related,
as the attribute that causes the false positive has a path of
`rustc_copy_clone_marker`.

Fixes #2475

[that_comment]: https://github.com/rust-lang/rust/issues/35900#issuecomment-245978831

Merge pull request #2575 from rust-lang-nursery/oli-obk-patch-1

Update needless_lifetimes_impl_trait.rs

Update needless_lifetimes_impl_trait.rs

Merge pull request #2574 from mark-i-m/i128

i128 is stabilizing

whitespace

attempt fix stderr

make it pass for now

i128 is stable

i128 is stable

Merge pull request #2570 from CYBAI/update-readme

Update configuration for leading dot filename

Update configuration for leading dot filename

Merge pull request #2569 from rust-lang-nursery/oli-obk-patch-1

Undo a temporary fix for a cargo bug