Refactor: Cleanup one part of assign_ops lint

Removes a lot of indentation and separates lint emission from lint
logic. Only touches the `hir::ExprKind::AssignOp` part of the lint.

Update comment location

Auto merge of #3856 - mikerite:clippy-dev-enchancement-1, r=phansch

Improve Clippy dev help

+ Print help if no subcommand is supplied
+ Make a short version of `update_lints` help for the subcommand listing

Auto merge of #3855 - rchaser53:issue-3849, r=flip1995

Remove an unused binary file

related https://github.com/rust-lang/rust-clippy/pull/3853
I'm sorry.

Improve Clippy dev help

+ Print help if no subcommand is supplied
+ Make a short version of `update_lints` help for the subcommand listing

Remove an unused binary file

align with rust-lang/rust/#58992

Auto merge of #3853 - rchaser53:issue-3849, r=flip1995

fix missing a semicolon

related: https://github.com/rust-lang/rust-clippy/issues/3849

fix missing a semicolon

Refactor: Extract `trait_ref_of_method` function

Auto merge of #3850 - mikerite:rustfmt-tests, r=phansch

More test improvements

Enable rustfix on `unused_unit` tests

Add `[rustfmt::skip]` to `unused_unit` test

Add `[rustfmt::skip]` to `doc_markdown` tests

Auto merge of #3803 - felix91gr:master, r=oli-obk

Cognitive Complexity (step 1 out of 3+): name changes

Following up on #3793

**Overall checklist:**

1. **Name changes**
2. MVP of functionality
3. Tests

After this PR, we will start working on the implementation itself.

Renamed: Cyclomatic Complexity -> Cognitive Complexity

* Ran automatic naming update

* Formalized rename of `cyclomatic_complexity` to `cognitive_complexity`
** Added the rename to `lib.rs`
** Added rename test

* Added warning for deprecated key `cyclomatic_complexity_threshold` and tests for it

* Added deprecation status for Clippy's builtin attribute

* Updated tests for new builtin attribute renaming

Auto merge of #3845 - euclio:unused-comments, r=phansch

move lint documentation into macro invocations

This PR moves lint documentation inside `declare_clippy_lint!` macro invocations, to avoid triggering the `unused_doc_comments` lint once it's modified in rust-lang/rust#57882. This PR is necessary to unblock that work, since the large number of warnings generated in `clippy_lints` causes Travis to hit the log length limit.

This PR also updates the documentation and website generation script.

It would be nice to get a clippy update in the Rust repo once this is merged.

cc @phansch

Auto merge of #3846 - mikerite:fix-3839, r=phansch

Ensure `expect_fun_call` bad suggestion is fixed

Closes #3839

Ensure `expect_fun_call` bad suggestion is fixed

Closes #3839

fix or ignore failing doc tests

move lint documentation into macro invocations

Auto merge of #3844 - phansch:const_fn_external_macro, r=oli-obk

Fix two missing_const_for_fn false positives

Fixes #3841

* Fixes false positive in external macros
* Fixes false positive when implement trait methods

Fix missing_const_for_fn for impl trait methods

Don't trigger missing_const_for_fn in external macros

As reported in #3841. Only fixes the part where it triggers on the
`derive`.

Add rustc nightly docs to cheatsheet

s/you/we; One more checklist item

Auto merge of #3838 - mati865:rustup, r=phansch

Upgrade compiletest_rs to 0.3.19

Upgrade compiletest_rs to 0.3.19

Add the rustc guide to the cheatsheet

More text fixes after Test section was moved to top

Reword 'Setup' a bit

Note that stderr files don't have to be created manually

Author lint usage via playground

Run remark in `doc` directory, too.

Cleanup 'Writing code' in CONTRIBUTING.md

This is now all included in `doc/adding_lints.md`

Copy from CONTRIBUTING.md: author lint, rustfix tests, rustfmt

And adapt the wording a bit so that it fits in the overall guide.

Mention Clippy utils

Remind to run update-all-references.sh at the end

Use positive if condition for readability

Expand on lint implementation section, wrap lines

Implicit return

Co-Authored-By: phansch <dev@phansch.net>

Implicit return

Co-Authored-By: phansch <dev@phansch.net>

Add lint writing documentation

This adds a new documentation page that explains how to write Clippy
lints. It guides the reader through creating a `foo` function lint.

Auto merge of #3834 - ljedrz:HirIdification_fix, r=phansch

HirIdification fixes

Supersedes https://github.com/rust-lang/rust-clippy/pull/3828, enables https://github.com/rust-lang/rust/pull/58836.

As usual, requesting a branch.

fmt fixes

align with rust-lang/rust/#58836

enable rust-lang/rust#58754

Auto merge of #3823 - spastorino:place_base, r=flip1995

Place::Local(x) is now Place::Base(PlaceBase::Local(x))

We need to merge this after the beta cut for this https://github.com/rust-lang/rust/pull/58631 to work.
/cc @oli-obk

Place::Local(x) is now Place::Base(PlaceBase::Local(x))

Auto merge of #3833 - mati865:rustup, r=oli-obk

Fix build with the master

https://github.com/rust-lang/rust/pull/58689 broke compiletest.

Use temporary branch for compiletest

https://github.com/rust-lang/rust/pull/58689 broke it

Auto merge of #3831 - martin-t:master, r=flip1995

Use the same <title> as <h1> on gh-pages

I think this makes the page easier to find using firefox's URL bar - it (afaik) searches based on the page's URL and title so if somebody doesn't have it bookmarked but remembers that the page contained "ALL lints" or something similar it's easy to go to it directly without resorting to a search engine.

Auto merge of #3830 - flip1995:unknown_attribute, r=oli-obk

Error on unknown or deprecated attributes

The reason I decided to not make this a lint is, that unknown attributes are always a hard error in rustc and also in rustfmt, which cannot be turned off by an `allow`. This could have been done by introducing a category which would be forbid-by-default. But that would have introduced a new lint category+group for just one lint and also wouldn't be consistent with rustfmt and rustc.

Add test for unknown Clippy attributes

Use the same <title> as <h1> on gh-pages

Error an unknown or deprecated Clippy attribute

Auto merge of #3829 - mikerite:fix-double-parens-suggestions, r=flip1995

Add `rustfmt::skip` to `double_parens` tests

Add `rustfmt::skip` to `double_parens` tests

Auto merge of #3666 - detrumi:map-or-on-non-copy, r=flip1995

Only suggest map_or for copy types

Fixes #2686

Auto merge of #3826 - mikerite:rustfmt-outer-attr-tests, r=phansch

Add `rustfmt::skip` to outer attr format tests

Auto merge of #3827 - phansch:rustup, r=Manishearth

Rustup rust-lang/rust#58321

Fixes the build after https://github.com/rust-lang/rust/issues/58321

Rustup https://github.com/rust-lang/rust/pull/58321

Add `rustfmt::skip` to outer attr format tests

Auto merge of #3821 - g-bartoszek:redundant_closure-different-borrow-levels, r=oli-obk

do not trigger redundant_closure when there is a difference in borrow…

… level between closure parameter and "self", fixes  #3802

Auto merge of #3818 - phansch:update_changelog, r=oli-obk

Update changelog for Rust 1.33

None

Fix false negative

Attempt to fix false negative

Move lint_map_unwrap_or to its own file

Only suggest map_or for copy types

Auto merge of #3817 - rust-lang:fix-bool_comparison-on-non-bool, r=oli-obk

Fix `bool_comparison` with non-`bool` expressions

Fixes #3703.
It just moves around the type check that was already there for some comparison to all of them, because if one type isn't `bool`, none of those comparison can be simplified.

do not trigger redundant_closure when there is a difference in borrow level between closure parameter and "self"

Auto merge of #3814 - ljedrz:HirIdification_lockstep_upgrade, r=phansch

HirIdify some lints

Unblocks https://github.com/rust-lang/rust/pull/58561 (a part of [rust-lang/rust#57578](https://github.com/rust-lang/rust/pull/57578)). Can we branch it like with https://github.com/rust-lang/rust-clippy/pull/3790? I can rebase on a different commit if need be.

Haven't had time to run tests yet, so I'd wait for Travis 🙈.

fix line format

Auto merge of #3819 - mikerite:rustfmt-formatting-tests, r=phansch

Add [rustfmt::skip] to formatting lint tests

Auto merge of #3820 - mikerite:remove-feature-try-from, r=Manishearth

Remove `#[feature(try_from)]`

`try_from` is now stable.

Remove `#[feature(try_from)]`

`try_from` is now stable.

Add [rustfmt::skip] to formatting lint tests

Update changelog for Rust 1.33

Fix `bool_comparison` with non-`bool` expressions

Auto merge of #3808 - mikerite:useless-format-suggestions, r=oli-obk

Fix `useless_format` suggestions

Auto merge of #3816 - g-bartoszek:redundant_closure_for_macros, r=Manishearth

do not trigger redundant_closure for external macros

fixes #3791

redundant_closure does not trigger for external macros

Auto merge of #3772 - flip1995:ice-3719, r=Manishearth

Fix ICE #3719+#3718 in lint match_ref_pats

Fixes #3719
This conveniently also fixes #3718

The ICE occurs when the match expression was a macro call, where the macro was defined in another file. Since we don't have the ability to reproduce this behavior with our UI tests (AFAIK), I couldn't add a test reproducing this ICE.. However, I added a test which is related to the ICE, to show the new behavior of the lint.

I tested it with the mscheme repo locally and the ICE didn't happen anymore.

r? @matthiaskrgr

Auto merge of #3805 - martinsp:ice-3747, r=Manishearth

Fix ICE #3747

I'm not sure if this was the correct approach.

I don't know if I put tests/ui/crashses/ice-3747.rs in correct place because the test always passed when I ran it with `cargo test`, even without the fix applied.

If I run that test with `env CLIPPY_TESTS=true cargo run --bin clippy-driver -- -L ./target/debug tests/ui/crashes/ice-3747.rs` then the test correctly fails without the fix applied

fixes #3747

fix: replace wrong id

Auto merge of #3815 - phansch:make_private, r=Manishearth

Make DiagnosticWrapper private

It's doesn't have to be public anywhere outside of clippy_lints as far as I can tell.

Make DiagnosticWrapper private

It's doesn't have to be public anywhere outside of clippy_lints

HirIdify some lints

Auto merge of #3811 - rust-lang:test-for-2526, r=mcarton

Add a test for #2526

Closes #2526, which seems to have been fixed at some point, but I couldn't find a test for it.

Add a test for #2526

Auto merge of #3810 - phansch:refactor/extract_module, r=flip1995

Extract diagnostics module and document some functions

This moves the lint building functions from `utils/mod.rs` to their own
`utils/diagnostics.rs` file. Also adds documentation for three of them.

Auto merge of #3790 - ljedrz:HirIdify_intravisit, r=phansch

partially HirIdify lints

Enables https://github.com/rust-lang/rust/pull/58232 (a part of https://github.com/rust-lang/rust/pull/57578).

fix formatting

Extract diagnostics module and document some functions

This moves the lint building functions from `utils/mod.rs` to their own
`utils/diagnostics.rs` file. Also adds documentation for three of them.

Document some `span_lint_*` util functions

Fix `useless_format` suggestions

Fix ICE #3747

[Martins Polakovs, John Firebaugh]

Auto merge of #3798 - eyko:patch-1, r=flip1995

Fix broken link in README

I noticed the link to the lint list had a typo, so this PR is just that:

```diff
- [lint list]](https://rust-lang.github.io/rust-clippy/master/index.html)
+ [lint list](https://rust-lang.github.io/rust-clippy/master/index.html)
```

Auto merge of #3801 - Wilfred:patch-1, r=Manishearth

Fix broken markdown in README.md

Fix broken markdown in README.md