add lint for mutable borrow; may have false positives. pushed for feedback

replace defids with nodeids for local variables

use def_id of function in check_for_mutation

refactor, add spans to warnings, add tests

Update test, fix lint

Rustup

attempt to add check for mutation of range bound within loop; compiles but doesn't work as intended. pushed for feedback

implement lint for mutable range bound

add lint declaration and example that should trigger the lint

Merge pull request #2060 from mrecachinas/feature/int-plus-one

Addresses #1955 - Suggests >= y + 1 become > y

Change rtype of int_plus_one detection to Option<String>

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

Remove old return-value

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

Rust upgrade to rustc 1.22.0-nightly (14039a42a 2017-09-22)

fix docs

Bump to 0.0.163

Update test expectations

Rust upgrade to rustc 1.22.0-nightly (14039a42a 2017-09-22)

Use span_lint_and_then as per feedback

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

Rust upgrade to rustc 1.22.0-nightly (0701b37d9 2017-09-18)

Bump to 0.0.162

Rust upgrade to rustc 1.22.0-nightly (0701b37d9 2017-09-18)

Fix case in doc_valid_idents

BibTeX and BibLaTeX use the usual capitalization of TeX tools:
- https://www.ctan.org/pkg/bibtex
- https://www.ctan.org/pkg/biblatex

Merge pull request #2066 from lucab/ups/doc-markdown

lints/doc_markdown: add two more entries

lints/doc_markdown: add two more entries

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 #2058 from rust-lang-nursery/ptr_arg-vs-clone

add suggestions for .clone() in ptr_arg fns

Merge pull request #2059 from marcusklaas/manual-slice-clone4

Also detect manual slice cloning

Extend MANUAL_MEMCPY lint so that it also detects manual clones between slices

Add test for manual slice clones

Format loops.rs with latest stable rustfmt

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

add suggestions for .clone() in ptr_arg fns

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