Refactor to use into_group_map from Itertools

s/direntry/dir_entry

Use insignificant whitespace mode for nice regex

cargo update in clippy_dev

update_lints rewrite: Add structure and --print-only

Merge pull request #3116 from phansch/fix_warning

Fix clippy -> clippy::all warning in CI

Fix clippy -> clippy::all warning in CI

Merge pull request #3115 from rust-lang-nursery/rm-token

Remove dependence of ci on github token

Remove dependence of ci on github token

Merge pull request #2941 from rust-lang-nursery/prs

Run travis on rustc-master instead of nightly

Merge pull request #3114 from nrc/pub-conf

Make `Conf::default` available

Make `Conf::default` available

Fixes RLS

Trigger rebuild for AppVeyor (and fix grammar)

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

Merge pull request #3113 from mikerite/fix-3112

Fix #3112

Fix #3112

Merge pull request #2977 from flip1995/tool_lints

Implement tool_lints

Fix ptr offset tests

declare_lint -> declare_tool_lint

Merge pull request #3081 from mikerite/fix-3078

Fix #3078

Merge branch 'master' into fix-3078

Revert "fix-3078: verify test case"

This reverts commit 6256ad05bac1f708298f827fdfa7b54042961294.

Merge pull request #2962 from phansch/further_automate_pre_publish

Further automate pre_publish.sh

Remove git diffing part

Because we no bump versions

Merge pull request #3076 from mbrubeck/patch-1

Remove incorrect note from string_add_assign docs

Merge pull request #3105 from frewsxcv/frewsxcv-private

Make clippy_lints::{utils,consts} modules private, remove unused items.

Implement backwards compatibility changes introduced by rust-lang/rust#53762

Fix some rebase fallout

Adapt codebase to the tool_lints

Adapt ui-toml-tests to the tool_lints

Adapt run-pass tests to the tool_lints

Adapt the *.stderr files of the ui-tests to the tool_lints

Adapt ui-tests to the tool_lints

Adapt scripts to the tool_lints

Adapt documentation to the tool_lints

Switch to declare_tool_lint macro

Switch to tool_lints

Merge pull request #3104 from frewsxcv/frewsxcv-ptr-offset-with-cast

New lint: Suggest `ptr.add([usize])` over `ptr.offset([usize] as isize)`.

dont deref

clippy suggestion

derive copy/clone

bring back sugg::range

eof newline

utilize carrier

suggest wrapping_offset as well

if no suggestion, dont add suggestion

utilize .is_unsafe_ptr

utilize cx.tcx.types.usize

tweak comment

switch lint from 'style' to 'complexity'

New lint: Suggest `ptr.add([usize])` over `ptr.offset([usize] as isize)`.

First part of #3047.

Make clippy_lints::{utils,consts} modules private, remove unused items.

Update imports and rustup

Merge pull request #3101 from pickfire/patch-1

Fix typo for panel cursor

Fix typo for panel cursor

Merge pull request #3090 from matthiaskrgr/new_without_default_highlight

docs: make example in new_without_default lint syntax highlighted

docs: make example in new_without_default lint syntax highlighted

Merge pull request #3018 from rust-lang-nursery/beta

We are riding the trains to stable, there's no point in trying to check the rustc version

Merge pull request #3087 from rust-lang-nursery/run

Use the compilers exit code computation instead of rolling our own

Use the compilers exit code computation instead of rolling our own

Merge pull request #3082 from mikerite/move-range-lints

Move some range lints to complexity

Merge pull request #3084 from mikerite/fix-3083

Remove `iterator_find_map` feature attribute

Remove `iterator_find_map` feature attribute

Closes #3083

Move some range lints to complexity

Recategorize `range_plus_one` and `range_minus_one` to `complexity`.
This moves `range_plus_one` out of the nursery as the inclusive range
syntax is now stable. Both are moved to `complexity` as it is more
consistent with other lints such as `int_plus_one`.

fix-3078: verify test case

Check the crash test case by commenting out the fix

Fix #3078

Merge pull request #3079 from matthiaskrgr/update_cargo_metadata

deps: update cargo_metadata from 0.5 to 0.6.

Merge pull request #3080 from matthiaskrgr/manual_swap_example

add how-to example for std::mem::transmute() usage instead of manual swap

add how-to example for std::mem::transmute() usage instead of manual swap

Merge pull request #3048 from goodmanjonathan/assign_op

deprecate assign_ops lint

Merge pull request #3059 from elpiel/writeln_empty_string_harcoded-suggestion

#3016 writeln_empty_string Hardcoded suggestion

deps: update cargo_metadata from 0.5 to 0.6.

Merge pull request #2924 from niklasf/copy-iterator

Add copy_iterator lint (#1534)

Add copy_iterator lint (#1534)

Merge pull request #3077 from matthiaskrgr/rustup_2

fix clippy breakage due to https://github.com/rust-lang/rust/pull/52602

fix clippy breakage due to https://github.com/rust-lang/rust/pull/52602

Remove incorrect note from string_add_assign docs

The docs claim that `String::push_str` is better than `String::add` because `String::add` allocates a new string and drops the old one, but this is not true.  In fact, `add` reuses the existing string and grows it only if its capacity is exceeded, exactly like `push_str`.  Their performance is identical since `add` is just a wrapper for `push_str`:

```
    fn add(mut self, other: &str) -> String {
        self.push_str(other);
        self
    }
```

https://github.com/rust-lang/rust/blob/35bf1ae25799a4e62131159f052e0a3cbd27c960/src/liballoc/string.rs#L1922-L1925

Merge pull request #3072 from matthiaskrgr/rustup

rustup, fix breakage introduced by rust-lang/rust#53581 and rust-lang/rust#53459

remove macro_vis_matcher feature gate since it is stable now.

Warning was:
warning: the feature `macro_vis_matcher` has been stable since 1.29.0 and no longer requires an attribute to enable
 --> src/lib.rs:4:12
  |
4 | #![feature(macro_vis_matcher)]
  |            ^^^^^^^^^^^^^^^^^
  |
  = note: #[warn(stable_features)] on by default

Remove now stable tool_attributes feature

rustup, fix breakage introduced by https://github.com/rust-lang/rust/pull/53581

Add LD_LIBRARY_PATH and GITHUB_TOKEN

Remove MinGW from CI

Remove unused code

Remove rust-toolchain file in CI

Keep the rustc master install in the travis file so we can use `travis_retry`

Try running appveyor on master instead of nightly

#3016 Add backticks for the msg

#3016 Add feedback and implement test for fixed hardcoded suggestion

#3016 [WIP] Implement feedback and suggestions

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

Rustup to rustc 1.30.0-nightly (6bf6d50a6 2018-08-19)

Remove implied rust_2018 feature

We are already on the edition and this feature is implied.

codemap -> source_map

https://github.com/rust-lang/rust/pull/52953

WIP of #3016 for hardocded suggestion for writeln on empty string

Merge pull request #3056 from mikerite/fix-2927

Fix #2927

fix-2927: Update formatting

Fix #2927

Rustup to rustc 1.30.0-nightly (23f09bbed 2018-08-14)

Merge pull request #3026 from dwijnand/fxhash

Add an internal lint for FxHashMap/FxHashSet

Merge pull request #3049 from mikerite/fix-2799

Add a test to ensure that #2799 is fixed