Merge remote-tracking branch 'upstream/master' into luckily_literals_love_lints

Loose ends and factor out consistency logic.

Use DigitInfo instead of get_digits().

do_lint() shouldn't format warnings.

Implement method for digit grouping suggestions

Add some structs for cleaning up lint logic later.

DigitInfo for storing radix, type, digits of a literal.
WarningType for making warning messages consistent.

rustfmt

Update from upstream.

Merge pull request #1909 from Arnavion/fix-manifest-path

Canonicalize --manifest-path argument before comparing it to cargo metadata.

Update .travis.yml

Bump the version

Update to latest master

Merge remote-tracking branch 'origin/master'

Canonicalize --manifest-path argument before comparing it to cargo metadata.

Before this change, a relative path like `--manifest-path=./Cargo.toml` would
fail to find a matching package in the cargo metadata. With this change,
both the argument and the cargo metadata path are canonicalized before
comparison.

Bump the version

Rustup gets rid of false positives

Merge pull request #1896 from Arnavion/fix-workspace

Fix logic that determines closest parent crate when invoked from a subdirectory.

Fix typo in previous commit

Wrong placement of inner attribute in previous commit

Rustc doesn't generate expansion info for `cfg!`

Fix logic that determines closest parent crate when invoked from a subdirectory.

The previous logic incorrectly matches the deepest child of the current
directory that is a crate.

Bump the version

Get rid of double double colons

Merge pull request #1899 from rust-lang-nursery/ui_test_changes

Fix ui tests

Merge pull request #1903 from kraai/fix-build

Set show_code_when_inline to true

Set show_code_when_inline to true

Fixes #1902

Use if let over match.

Fix lint descriptions.

Add lint for digit grouping.

Fix ui tests

Bump the version

Merge pull request #1883 from Manishearth/stash

Fix various needless_range_loop false positives

Fix path in cargo.toml, which was accepted by previous cargo versions

Fix various needless_range_loop false positives

Update rustc output

Merge pull request #1888 from kraai/fix-build

Use token_to_string instead of binop_to_string

Create LICENSE

Use token_to_string instead of binop_to_string

Fixes #1887

Run rustfmt

Bump the version

Rustup to *rustc 1.20.0-nightly (d84693b93 2017-07-09)*

Merge pull request #1871 from uHOOCCOOHu/never_loop_fix

Fix the bug remaining in #1586.

Merge pull request #1873 from uHOOCCOOHu/fix-tests

Fix tests.

Add more test for #1586.

Search `continue` in `return` expr.

Put tests for `step_by(0)` back.

Remove tests using removed feature `step_by`.
Update test outputs.

Merge pull request #1860 from Vurich/master

Allow cargo-clippy to work in subdirectories

ignore needless_lifetimes false positive

Merge pull request #1847 from Manishearth/nested_while_let

don't lint while_let_on_iterator on nested loops

Merge pull request #1852 from Manishearth/notrans

Use `--emit=metadata` instead of `-Zno-trans`

Bump the version

Merge pull request #1868 from messense/feature/fix-nightly-07-02

Fix compilation on rustc 1.20.0-nightly (067971139 2017-07-02)

Update windows ui tests

Update ui tests for examples

Update ui tests

Fix compilation on rustc 1.20.0-nightly (067971139 2017-07-02)

Merge pull request #1866 from Manishearth/oli-obk-patch-2

Update compiletest dependency

Update compiletest dependency

fix tests

Merge pull request #1862 from messense/feature/fix-nightly-06-28

Fix compilation on rustc 1.20.0-nightly (69c65d296 2017-06-28)

Run rustfmt

Bump the version

Fix compilation on rustc 1.20.0-nightly (69c65d296 2017-06-28)

Allow cargo-clippy to work in subdirectories

Merge pull request #1536 from CBenoit/master

Add lint `&ref x` patterns that could be just `x`

Merge pull request #1855 from Manishearth/for_loop

Fix for loop detection

Update for latest master

Use `--emit=metadata` instead of `-Zno-trans`

fixes #1500

Improve needless_borrowed_ref lint doc.

Add needless borrowed ref lint (WIP).

don't lint while_let_on_iterator on nested loops

The problem is with a nested loop, the iterator may well be reused. This
changeset introduces a false negative, when the iterator is initialized
within the outer loop. A further PR could get rid of this false negative
by checking if the iterator is indeed initialized within the outer loop.

Merge pull request #1841 from sanxiyn/span-lint-and-sugg

Use span_lint_and_sugg

Use span_lint_and_sugg

Merge pull request #1839 from Manishearth/rustup-tests

Fix the test suite after cargo update

Fix the test suite after cargo update

Merge pull request #1837 from Manishearth/step_by

Replace `Range::step_by` checking with `Iterator::step_by`

Replace `Range::step_by` checking with `Iterator::step_by`

Merge pull request #1799 from Manishearth/docs

Rewrite `doc_markdown` to use `pulldown-cmark`

Fix links in change log

Don't depend explicitly on `pulldown-cmark`

Use the one from `rustc`'s, which avoids "multiple matching crate"
errors.

Don't use default features for `pulldown-cmark`

Ignore mix of {,non-}sugared doc in `doc_markdown`

Add a new `doc_markdown` test from issue comments

Log that `doc_markdown` has been rewritten

I hope this will fix most false positives, and maybe some true
negatives, and won't introduce too much new FP.

Fix spans in all cases in `doc_markdown`

Fix spans in `doc_markdown` in most cases

Rewrite `doc_markdown` to use `pulldown-cmark`

Add `itertools` and `pulldown-cmark` dependencies

Bump the version

CHANGELOG is ordered after Cargo

Merge pull request #1833 from messense/feature/fix-nightly-06-16

rustup to rustc 1.19.0-nightly (258ae6dd9 2017-06-15)

Update ui tests

rustup to rustc 1.19.0-nightly (258ae6dd9 2017-06-15)

Merge pull request #1830 from scott-linder/generic-new-without-default

Ignore new-without-default lint when `new` method has generic types

Ignore new-without-default lint when `new` method has generic types

There may be no sensible `Default` impl if the result of `new` depends
on a type parameter.

Merge pull request #1501 from scott-linder/types-borrow-box

Types borrow box

Merge pull request #1827 from erickt/master

`in_macro` check in `NEEDLESS_RETURN`

`in_macro` check in `NEEDLESS_RETURN`

This allows you to define a macro like `try!(...)` that embeds an
early exit without emitting the "needless_return" warning.

Closes #1271.

Merge pull request #1826 from Manishearth/multi-zero-prefix

deal with multiple prefixed zeros in literals