Rustup

Apply applicability

Majority of PR changes

Lint against needless uses of `collect()`

Handles cases of `.collect().len()`, `.collect().is_empty()`, and
`.collect().contains()`. This lint is intended to be generic enough to
be added to at a later time with other similar patterns that could be
optimized.

Closes #3034

Merge pull request #3092 from illicitonion/issue-2879

default_trait_access skips <F as Default>::default()

Merge pull request #3120 from rust-lang-nursery/ci_integration

Update travis integration tests to tool_lints

Merge pull request #3119 from joshtriplett/a-message-from-the-save-the-iters-foundation

iter conservation efforts: save the endangered .iter() and .into_iter()

Update travis integration tests to tool_lints

`clippy_pedantic` -> `clippy::pedantic`
`clippy_nursery` -> `clippy::nursery`

iter conservation efforts: save the endangered .iter() and .into_iter()

Make explicit_iter_loop and explicit_into_iter_loop allow-by-default, so
that people can turn them on if they want to enforce that style; avoid
presenting them as *the* idiomatic Rust style, rather than just *a* style.

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

Merge branch 'master' into issue-2879

Simplify

Use types rather than strings

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

default_trait_access skips <F as Default>::default()

This includes the type name, so is clear, and may be necessary.

There doesn't seem to be an obviously cleaner way to pull out the
literal text of the named type here.

Fixes #2879

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.