Change rtype of int_plus_one detection to Option<String>

Change to returning Option<(bool, Option<String>)>

Remove old return-value

Use span_lint_and_then as per feedback

Add tests for 'int_plus_one'

Register 'int_plus_one' lint case in clippy_lints

(#1955): Suggests `x > y` over `x >= y + 1` for ints

This module handles the following cases:
- `... >= ... + 1` and `... >= 1 + ...`
- `... - 1 >= ...` and `-1 + ... >= ...`
- `... + 1 <= ...` and `... + 1 <= ...`
- `... <= ... - 1` and `... <= -1 + ...`

Note: this only goes 1 level deep (i.e., does not constant-fold) and
does not currently simplify expressions. Examples of these
cases include:
```rust
let x = 1;
y >= y + x; // won't catch this case or any permutation

x + 1 >= y + 2; // won't catch this case

x + 1 - 1 >= y - 1 + 1; // WILL catch this case when it likely shouldn't
```

Merge pull request #2057 from topecongiro/issue-1818

Enhance CHARS_*_CMP lint

Merge pull request #2056 from topecongiro/issue-1851

Add suggestion to needless_borrow

Update tests

Enhance CHARS_*_CMP lint

Update tests

Add suggestion to needless_borrow

Merge pull request #2052 from topecongiro/issue-1990

Add ExprLoop to contains_continue_expr()

Update a test

Add ExprLoop to contains_continue_expr()

Bump version

Version bump

Merge pull request #2049 from topecongiro/rustc-fix

rustc fix

Remove '\t' from .stderr

to make cargo test pass

Add a missing argument

Merge remote-tracking branch 'origin/master' into rustup

Rustup

Merge pull request #2046 from siiptuo/small-doc-fixes

Small documentation fixes

Fix link in trivial_regex

Fix empty documentation in unit_expr

Fix links in approx_const

Version bump

Rustup

Regressions (#2041)

Merge pull request #2039 from rust-lang-nursery/ptr-arg-suggestion

suggestion for ptr_arg

suggestion for ptr_arg

Merge pull request #2037 from Aaron1011/clone-rc

Add CLONE_ON_REF_PTR lint

Add CLONE_ON_REF_PTR lint

Closes issue #1645

Merge pull request #2034 from Aaron1011/update-rust

Update for latest Rust

Bump version

Use hir_id instead of fetching hir_id via the NodeId

Update for latest Rust

This is mainly due to https://github.com/rust-lang/rust/commit/dead08cb33134

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

Soft rustup (only fixed some tests)

Soft rustup (only fixed some tests)

Merge pull request #1963 from rust-lang-nursery/upstream

Rust needs clippy to have a Cargo.lock

Update changelog

Merge pull request #2004 from sunfishcode/master

When suggesting `from(x)` for lossless casts, strip parens from `x`.

format comment

Merge pull request #2023 from montrivo/while_let_loop

`while_let_loop` doesn't take into account break-with-value #1948

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

Also ignore `continue` statements in `is_unit_expr`

`while_let_loop` doesn't take into account break-with-value #1948

Also ignore `continue` statements in `is_unit_expr`

for loops -> for-loops

Only strip parens for binary expressions.

Add a testcase demonstrating how precedence interacts with the lossless-cast lint.

When suggesting `from(x)` for lossless casts, strip parens from `x`.

Update changelog

Update changelog

Merge pull request #2021 from marcusklaas/needless-loop-2

Add lint to detect manual slice copies

Add lint to detect manual slice copies

Rustfmt for_loop.rs and add false positive tests

Run nightly rustfmt

Merge pull request #2020 from 0xbsec/patch-1

Fix "further information" link

Fix "further information" link

"further information" link was missing the `v` part from the url.
e.g.
wrong (404 notfound): https://rust-lang-nursery.github.io/rust-clippy/0.0.157/index.html#map_entry
correct: https://rust-lang-nursery.github.io/rust-clippy/v0.0.157/index.html#map_entry

Version Bump

Fix `len_zero` ICE

Update to latest nightly

Fix dogfood

Run rustfmt

Fix unit_expr expectations and changelog entry

Merge branch 'pr-2016' into HEAD

Mention the false positive

Bump to 0.0.156

Rustup to rustc 1.22.0-nightly (744dd6c1d 2017-09-02) (fixes #2013)

Rustfmt tests

Fix all the clippy lints

Add false positive tests

Remove direct call for Deref

Remove "assigns" from the lint

RustFmt file and tests

Don't trigger lint on break or return

More relevant tests

Improve the lint message

Update unit tests

early tests

Fix false positives in assignment inside the else condition

RustFmt changes

Switch back to manual deref

Improved spans for lints and support match expressions

Handle method calls

Fix brace indentation

Check method calls

More initial work

Introduce check_stmt

Use the type from the macro

Initial commit of unit expr

Merge pull request #2010 from rust-lang-nursery/doc_improvements

some small doc improvements

Merge pull request #2009 from rust-lang-nursery/mcarton-patch-1

Move the number of lints back to the top of README

some small doc improvements

Move the number of lints back to the top of README

This used to be at the top and was moved at the bottom when the big list of lints started to be so ridiculously long that people had to scroll for 10 minutes to have usage information :smile:

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

Remove clippy.bashy.io badge

Remove clippy.bashy.io

The service seems to be defunct for a while now

Merge pull request #2007 from rust-lang-nursery/wiki

Deprecate the wiki and remove the lint list from the README (fixes #1933)

Update README lint counter message

Link to current versions docs instead of master docs

wiki -> docs