Merge pull request #458 from sanxiyn/rustfmt

rustfmt a little

rustfmt a little

Merge pull request #457 from sanxiyn/match-block-comma

Remove trailing commas in match arms with blocks

Remove trailing commas in match arms with blocks

Merge pull request #452 from fhartwig/lifetime-false-positives

Fix some false positive in needless_lifetimes lint

Merge pull request #454 from apasel422/issue-446

Match `min` and `max` functions using `DefId`

Match `min` and `max` functions using `DefId`

Closes #446.

Merge pull request #453 from apasel422/transmute

Add lint for useless transmutes

Add lint for useless transmutes

Closes #441.

Add some tests for lifetime elision lint with types and traits with lifetimes

Track elided lifetimes in types and trait objects

Merge pull request #450 from Manishearth/mir_float_cmp

use rustc's eval_const, bail on (negative) infinity

use rustc's eval_const, bail on (negative) infinity

Merge pull request #449 from Manishearth/shadow_visitor

use visitor for contains_self

use visitor for contains_self

Merge pull request #447 from apasel422/executable

Remove executable permission from remaining source files

Remove executable permission from remaining source files

Merge branch 'pr-445'

update readme to specify both lint groups are needed for the  whole set

As a new user and newbie to rust, after reading this part of the readme("whole set"), I thought I was seeing a bug when I used `#![deny(clippy)]` but also had to add `#![deny(shadow_unrelated)]`.
But this explained it: https://github.com/Manishearth/rust-clippy/blob/3322ffa8a048ef5369d3cdd914869fdf383473a4/src/lib.rs#L108

Tests shouldn't be executable (fixes #444)

Merge pull request #440 from Manishearth/map_clone

match .map(Clone::clone)

match .map(Clone::clone)

Rustup to rustc 1.6.0-nightly (effcd2965 2015-11-04)

fixes #437

Merge pull request #435 from sanxiyn/temporary-assignment

New lint for assignment to temporary

New lint for assignment to temporary

Merge pull request #427 from wartman4404/master

Prefer `.cloned()` over `.map(|x| x.clone())`

check for Deref conversions

New lint for using `.cloned()`

Merge pull request #432 from nweston/zip-with-range

New lint for zip with array length instead of enumerate()

New lint for zip with array length instead of enumerate()

Fixes #11.

Merge pull request #426 from Manishearth/fix-ptr-arg

Fix ptr-arg false positive for trait impls

Fix ptr-arg false positive for trait impls

Fixes #425

Merge pull request #422 from sanxiyn/no-effect

New lint for statement with no effect

New lint for statement with no effect

Merge pull request #423 from wartman4404/master

Duplicate warnings for match_bool / single_match #403

Don't show single_match if match_bool also applies

Merge pull request #396 from fhartwig/while-let-for

Suggest for loop instead of while-let when looping over iterators

Allow needless_lifetime to pass dogfood.sh

Fix false positives when iterator variable is used after the loop

Fix error in README

Fixes for code review comments

* remove weird infinite loops from compile-tests
* remove call to Option::unwrap
* in the lint message, show while-let loop rewritten as for loop

Don't suggest using a for loop if the iterator is used in the loop body

Due to https://github.com/rust-lang/rust/issues/8372, we have to use while-let
in these cases.

Suggest for loop instead of while-let when looping over iterators

Merge pull request #415 from birkenfeld/fix_409

Fix reverse_range_loop not taking sign into account (fixes #409)

Fix reverse_range_loop not taking sign into account (fixes #409)

Adds a Display impl for Constant, because that might come in handy
elsewhere as well.

fix

bump version, add optional dep info

Merge pull request #413 from iKevinY/approx-const

Compare float literals to stringified constants

Compare float literals to stringified constants

- Convert constants to strings and compare directly with float literal.
- Return immediately after positive match for constant.
- Fix value of `my_log10_e` in `approx_const` tests.

Merge pull request #408 from iKevinY/approx-const

Change implementation of approx_const lint

Change implementation of approx_const lint

- Replace epsilon with lower and upper bounds for each constant.
- Warn on use of "3.14", and update tests accordingly.

Merge pull request #405 from sanxiyn/needless-update

New lint for struct update that has no effect

New lint for struct update that has no effect

Merge pull request #402 from Manishearth/match_bool_hints

added code snippet help to match_bool

added code snippet help to match_bool

Merge pull request #401 from kvikas/match_bool

Match on bool should be replaced with if..else block

Fixups from review comments

1. Moved common check `in_external_macro` to the top of function from inside each
conditionals.
2. Inlined `is_bool_expr` call

Ran util/update_lints.py to auto gen doc and lib.rs

Match on bool should be replaced with if..else block

1. Added another conditional in `check_expr` impl to lint if match expr
is a bool.
2. Test cases.

Merge branch 'pr-394'

Cleanup as discussed in PR

Make is_from_for_desugar sound (rust/28973 got fixed)

Regnerate README.md

New lint for needless use of nightly features

bump again

Fix type complexity lint

Merge pull request #395 from Manishearth/rustup

rustup

Merge branch 'master' of https://github.com/Manishearth/rust-clippy into rustup

workaround for failing test

bump crate version

rustup

bump crates

Merge pull request #392 from fhartwig/warn-ptr-args

Make ptr_arg lint warn by default

Make ptr_arg lint warn by default

Merged #365

Merged #366

Merge PR #374

Merge pull request #382 from Manishearth/empty_loop

new lint to detect `loop {}`

update lints

new lint to detect

Merge pull request #390 from Manishearth/precedence

improved precedence messages (fixes #389)

improved precedence messages (fixes #389)

Improve cmp_owned suggestions (fixes #386)

whup the while_let_loop for ignoring expressions!

Merge pull request #379 from swgillespie/zero_div_zero_nan

implement 0.0/0.0 -> NaN lint as described in #370

Merge pull request #381 from durka/patch-2

fix doc comment for if_let_chain!

Remove "are you sure?" from lint msg.

No added value, and leads to punctuation clash.

fix doc comment for if_let_chain!

implement 0.0/0.0 -> NaN lint as described in #370

casing of NaN

Merge pull request #377 from Manishearth/rustup

Rust upgrade to rustc 1.5.0-nightly (9d3e79ad3 2015-10-10)

Rust upgrade to rustc 1.5.0-nightly (9d3e79ad3 2015-10-10)

Recommend using Mutex<()> for locking

Merge pull request #369 from fhartwig/allow-warn

Fix documentation that disagrees with code

Fix documentation that disagrees with code

Add a lint for sized integer types in a mutex

Change lint description

Add "nonsensical OpenOptions" lint

Suggest using an atomic value instead of a Mutex where possible

Merge pull request #354 from Pyriphlegethon/master

Add "unnecessary mut passed" lint

RingBuf was renamed to VecDeque (fixes #363)