]>
git.lizzy.rs Git - rust.git/log
Matthias Krüger [Thu, 26 Sep 2019 16:34:43 +0000 (18:34 +0200)]
rustup https://github.com/rust-lang/rust/pull/64515
changelog: none
bors [Thu, 26 Sep 2019 10:33:44 +0000 (10:33 +0000)]
Auto merge of #4581 - Manishearth:osx-enable, r=matthiaskrgr
Re enable OSX builders
Fixes #4576
changelog: none
Manish Goregaokar [Thu, 26 Sep 2019 10:31:06 +0000 (03:31 -0700)]
Re enable OSX builders
Fixes #4576
bors [Thu, 26 Sep 2019 09:13:59 +0000 (09:13 +0000)]
Auto merge of #4580 - lzutao:rustup, r=flip1995
Rustup rust-lang/rust#64513
changelog: none
Lzu Tao [Thu, 26 Sep 2019 09:04:45 +0000 (16:04 +0700)]
Remove unused import
Lzu Tao [Thu, 26 Sep 2019 09:03:36 +0000 (16:03 +0700)]
Rustup https://github.com/rust-lang/rust/pull/64513
bors [Thu, 26 Sep 2019 08:06:16 +0000 (08:06 +0000)]
Auto merge of #4568 - mikerite:fix-4548, r=flip1995
Fix `nonminimal-bool` false positive
Closes #4548
Closes #3847
changelog: Fix `nonminimal-bool` false positive
bors [Thu, 26 Sep 2019 07:38:08 +0000 (07:38 +0000)]
Auto merge of #4569 - james9909:add-comparison-chain, r=oli-obk
Add a new lint for comparison chains
changelog: Adds a new lint: `comparison_chain`.
`comparison_chain` lints all `if` conditional chains where all the conditions are binary comparisons on the same two operands and will suggest a rewrite with `match`.
Closes #4531.
bors [Thu, 26 Sep 2019 06:02:21 +0000 (06:02 +0000)]
Auto merge of #4570 - lzutao:call_site_toplevel_ref_arg, r=phansch
Fix macro expansion in toplevel_ref_arg lint
changelog: Fix macro expansion in toplevel_ref_arg lint
Lzu Tao [Thu, 26 Sep 2019 01:53:39 +0000 (01:53 +0000)]
Remove unused attribute in test
Lzu Tao [Thu, 26 Sep 2019 01:46:51 +0000 (08:46 +0700)]
Fix macro expansion in toplevel_ref_arg lint
Lzu Tao [Wed, 25 Sep 2019 04:06:52 +0000 (11:06 +0700)]
Add regression test for macro expansion
bors [Wed, 25 Sep 2019 21:54:14 +0000 (21:54 +0000)]
Auto merge of #4575 - Manishearth:suggestions, r=oli-obk
Make more tests rustfixable
Fixes https://github.com/rust-lang/rust-clippy/issues/3630
changelog: Improve suggestions for many lints in preparation for `cargo fix --clippy`
r? @phansch @yaahc
Manish Goregaokar [Wed, 25 Sep 2019 21:53:20 +0000 (14:53 -0700)]
Downgrade op_ref to a MaybeIncorrect suggestion
Manish Goregaokar [Wed, 25 Sep 2019 17:49:56 +0000 (10:49 -0700)]
unnecessary_operation: make test rustfixable
Manish Goregaokar [Wed, 25 Sep 2019 17:46:14 +0000 (10:46 -0700)]
unnecessary_clone: split rustfixable lint out into separate test
Manish Goregaokar [Wed, 25 Sep 2019 17:33:48 +0000 (10:33 -0700)]
string_add, string_add_assign: split tests, make one rustfixable
Manish Goregaokar [Wed, 25 Sep 2019 17:21:08 +0000 (10:21 -0700)]
redundant_static_lifetimes: split test, make rustfixable
Manish Goregaokar [Wed, 25 Sep 2019 17:08:19 +0000 (10:08 -0700)]
renamed_builtin_attr: make test rustfixable
Manish Goregaokar [Wed, 25 Sep 2019 17:01:21 +0000 (10:01 -0700)]
redundant_pattern_matching: make rustfixable
Manish Goregaokar [Wed, 25 Sep 2019 16:30:27 +0000 (09:30 -0700)]
redundant_closure_call: split tests into fixable
Manish Goregaokar [Wed, 25 Sep 2019 16:03:32 +0000 (09:03 -0700)]
map_unit_fn: make test rustfixable
Manish Goregaokar [Wed, 25 Sep 2019 15:49:23 +0000 (08:49 -0700)]
map_unit_fn: fix applicability
Manish Goregaokar [Wed, 25 Sep 2019 15:48:09 +0000 (08:48 -0700)]
option_map_unit_fn: Split into fixable/unfixable
Manish Goregaokar [Wed, 25 Sep 2019 15:39:45 +0000 (08:39 -0700)]
map_unit_fn: rename tests to fixable
Manish Goregaokar [Wed, 25 Sep 2019 12:39:56 +0000 (05:39 -0700)]
non_copy_const: remove incorrect suggestion
Manish Goregaokar [Wed, 25 Sep 2019 12:28:19 +0000 (05:28 -0700)]
needless_return: add allow()s to test, make rustfixable
Manish Goregaokar [Wed, 25 Sep 2019 12:25:31 +0000 (05:25 -0700)]
needless_collect: fix suggestion, make test rustfixable
Manish Goregaokar [Wed, 25 Sep 2019 12:19:09 +0000 (05:19 -0700)]
needless_borrowed_ref: fix false positive, make rustfixable
Manish Goregaokar [Wed, 25 Sep 2019 12:03:28 +0000 (05:03 -0700)]
needless_borrow: allow other lints, make fixable
Manish Goregaokar [Wed, 25 Sep 2019 11:59:10 +0000 (04:59 -0700)]
mem_discriminant: split test, make rustfixable
Manish Goregaokar [Wed, 25 Sep 2019 11:50:23 +0000 (04:50 -0700)]
map_flatten: make it a rustfix test
Manish Goregaokar [Wed, 25 Sep 2019 11:49:38 +0000 (04:49 -0700)]
Leave note on non-rustfixable tests
Manish Goregaokar [Wed, 25 Sep 2019 11:46:43 +0000 (04:46 -0700)]
Remove large-digit-groups test from literals.rs
Manish Goregaokar [Wed, 25 Sep 2019 11:38:17 +0000 (04:38 -0700)]
map_entry test: Fix semicolon, add run-rustfix
Manish Goregaokar [Wed, 25 Sep 2019 11:31:46 +0000 (04:31 -0700)]
Split map_entry tests into fixable and unfixable
bors [Wed, 25 Sep 2019 20:56:22 +0000 (20:56 +0000)]
Auto merge of #4574 - Manishearth:rustup, r=yaahc,centril
Rustup to rustc 1.39.0-nightly (
acf7b50c7 2019-09-25)
changelog: none
fixes https://github.com/rust-lang/rust/issues/64777
r? @phansch @yaahc
Manish Goregaokar [Wed, 25 Sep 2019 11:29:39 +0000 (04:29 -0700)]
Remove suggestion for complex map_entry cases
Manish Goregaokar [Wed, 25 Sep 2019 20:51:29 +0000 (13:51 -0700)]
allow osx failures
Manish Goregaokar [Wed, 25 Sep 2019 20:34:55 +0000 (13:34 -0700)]
ignore single-match for or patterns
Manish Goregaokar [Wed, 25 Sep 2019 19:00:17 +0000 (12:00 -0700)]
arm.pats -> arm.pat
Manish Goregaokar [Wed, 25 Sep 2019 11:16:25 +0000 (04:16 -0700)]
Rustup to rustc 1.39.0-nightly (
acf7b50c7 2019-09-25)
- Addresses inference error
- Updates compiletest
Michael Wright [Wed, 25 Sep 2019 04:20:40 +0000 (06:20 +0200)]
Refactor `booleans`
Inline `snip (..)` function
James Wang [Wed, 25 Sep 2019 00:07:03 +0000 (19:07 -0500)]
Update refactor according to code review
James Wang [Tue, 24 Sep 2019 22:05:43 +0000 (17:05 -0500)]
Fix example in docs
James Wang [Tue, 24 Sep 2019 21:55:05 +0000 (16:55 -0500)]
Add a new lint for comparison chains
Michael Wright [Tue, 24 Sep 2019 06:06:58 +0000 (08:06 +0200)]
Fix `nonminimal-bool` false positive
Closes #4548
Closes #3847
bors [Mon, 23 Sep 2019 17:01:13 +0000 (17:01 +0000)]
Auto merge of #4567 - phansch:toplevel_ref_arg, r=flip1995
Add run-rustfix for toplevel_ref_arg lint
changelog: none
cc #3630
Philipp Hansch [Mon, 23 Sep 2019 09:19:24 +0000 (11:19 +0200)]
Add run-rustfix for toplevel_ref_arg lint
bors [Mon, 23 Sep 2019 08:25:38 +0000 (08:25 +0000)]
Auto merge of #4561 - rust-lang:let-return-fix, r=phansch
Fix let_and_return lint
This fixes #4555 (false positive for the `let_and_return` lint).
changelog: none
Andre Bogus [Fri, 20 Sep 2019 16:20:49 +0000 (18:20 +0200)]
Fix let_and_return lint
bors [Mon, 23 Sep 2019 07:11:21 +0000 (07:11 +0000)]
Auto merge of #4566 - phansch:update_changelog2, r=flip1995
Update the changelog for Rust 1.38
[Rendered](https://github.com/phansch/rust-clippy/blob/update_changelog2/CHANGELOG.md#rust-138)
changelog: none
bors [Mon, 23 Sep 2019 05:51:04 +0000 (05:51 +0000)]
Auto merge of #4537 - mikerite:unneeded_wildcard_pattern, r=phansch
Add `unneeded-wildcard-pattern` lint
changelog: Add `unneeded-wildcard-pattern` lint
Michael Wright [Mon, 23 Sep 2019 03:26:47 +0000 (05:26 +0200)]
Merge branch 'master' into unneeded_wildcard_pattern
Philipp Hansch [Sun, 22 Sep 2019 17:12:42 +0000 (19:12 +0200)]
Update the changelog for Rust 1.38
bors [Sun, 22 Sep 2019 11:40:35 +0000 (11:40 +0000)]
Auto merge of #4518 - imp:4517_license-file, r=phansch
clippy::cargo_common_metadata: check for license-file
When license property is missing in Cargo.toml check for license-file
as it may be used instead of the former. The check implemented here is
very naive as it only verifies that the field is present and is not
empty. More scrutiny can be applied by verifying the file is actually
present.
Fixes #4517
changelog: clippy::cargo_common_metadata now checks for license-file when license is missing
bors [Sun, 22 Sep 2019 10:36:32 +0000 (10:36 +0000)]
Auto merge of #4565 - matthiaskrgr:rustup_16, r=phansch
rustup https://github.com/rust-lang/rust/pull/64666
changelog: none
Matthias Krüger [Sun, 22 Sep 2019 10:35:20 +0000 (12:35 +0200)]
rustup https://github.com/rust-lang/rust/pull/64666
Cyril Plisko [Sat, 7 Sep 2019 08:00:02 +0000 (11:00 +0300)]
clippy::cargo_common_metadata: check for license-file
When license property is missing in Cargo.toml check for license-file
as it may be used instead of the former. The check implemented here is
very naive as it only verifies that the field is present and is not
empty. More scrutiny can be applied by verifying the file is actually
present.
Fixes #4517
Michael Wright [Sun, 22 Sep 2019 07:10:39 +0000 (09:10 +0200)]
Add additional tests to unneeded_wildcard_pattern
Michael Wright [Sun, 22 Sep 2019 06:59:23 +0000 (08:59 +0200)]
Merge branch 'master' into unneeded_wildcard_pattern
bors [Sat, 21 Sep 2019 11:42:15 +0000 (11:42 +0000)]
Auto merge of #4562 - phansch:wildcard_enum_match_rustfix, r=llogiq
Add run-rustfix for wildcard_enum_match_arm lint
changelog: none
cc #3630
Philipp Hansch [Sat, 21 Sep 2019 05:26:18 +0000 (07:26 +0200)]
Add run-rustfix for wildcard_enum_match_arm lint
bors [Sat, 21 Sep 2019 01:01:21 +0000 (01:01 +0000)]
Auto merge of #4558 - Manishearth:suggestions, r=phansch
Make more tests rustfixable
changelog: Fix various lint suggestions
Progress towards https://github.com/rust-lang/rust-clippy/issues/3630
r? @phansch
Manish Goregaokar [Fri, 20 Sep 2019 07:40:20 +0000 (16:40 +0900)]
inline_fn_without_body: make it use a rustfix page
Manish Goregaokar [Fri, 20 Sep 2019 07:07:13 +0000 (16:07 +0900)]
implicit_return: make it use a rustfix test
Manish Goregaokar [Fri, 20 Sep 2019 06:53:58 +0000 (15:53 +0900)]
identity_conversion: make it use a rustfix test
Manish Goregaokar [Fri, 20 Sep 2019 06:25:05 +0000 (15:25 +0900)]
for_loop: Split test into fixable/unfixable, make needless_range_loop use updated range syntax
Manish Goregaokar [Fri, 20 Sep 2019 06:16:33 +0000 (15:16 +0900)]
eq_op: stop testing nonminimal_bool in same file
Manish Goregaokar [Fri, 20 Sep 2019 05:54:16 +0000 (14:54 +0900)]
op_ref: Move tests out of eq_op file
bors [Fri, 20 Sep 2019 07:27:36 +0000 (07:27 +0000)]
Auto merge of #4511 - rust-lang:replace_uninitialized, r=flip1995
New lint: mem_replace_with_uninit
changelog: add `mem_replace_uninit` lint
This fixes #4485
Manish Goregaokar [Thu, 19 Sep 2019 21:57:15 +0000 (06:57 +0900)]
deref_addrof_double_trigger: Note why rustfix can't be used
Manish Goregaokar [Thu, 19 Sep 2019 21:48:39 +0000 (06:48 +0900)]
assign_ops: Remove MachineApplicable
Andre Bogus [Fri, 13 Sep 2019 16:39:14 +0000 (18:39 +0200)]
new lint: mem-replace-with-uninit
bors [Thu, 19 Sep 2019 09:28:29 +0000 (09:28 +0000)]
Auto merge of #4544 - JoshMcguigan:issue-4542, r=flip1995
#4542 remove machine applicable suggestion
This helps #4542 (but does not completely resolve) by removing the machine applicable suggestion (which was incorrect) for that case.
I would have preferred to fix the machine applicable suggestion to handle format strings, but that's a bit beyond my current understanding of the clippy codebase. I'd be happy to give it a try given some guidance.
changelog: only produce machine applicable suggestions on `explicit_write` lint
bors [Thu, 19 Sep 2019 08:50:31 +0000 (08:50 +0000)]
Auto merge of #4539 - jolson88:cast-lossless-pedantic, r=flip1995
Changes cast-lossless to a pedantic lint
As discussed in #4528, this moves the cast-lossless lint from `all` to `pedantic`.
I couldn't tell from description alone if it should also be removed from the complexity category, so I left it as part of complexity for now. I didn't see any impact to the tests from this change, but I could be wrong (as this is my first PR).
fixes #4528
changelog: Moves cast-lossless from default to checking only as a `pedantic` lint.
bors [Thu, 19 Sep 2019 08:21:42 +0000 (08:21 +0000)]
Auto merge of #4535 - rust-lang:unsafe-doc, r=flip1995
New lint: Require `# Safety` section in pub unsafe fn docs
changelog: add `missing_safety_doc` lint
This fixes #2207
Andre Bogus [Wed, 11 Sep 2019 16:39:02 +0000 (18:39 +0200)]
New lint: Require `# Safety` section in pub unsafe fn docs
bors [Thu, 19 Sep 2019 04:12:43 +0000 (04:12 +0000)]
Auto merge of #4551 - mikerite:fix-ice-reporting, r=llogiq
Fix ice reporting
changelog: none
Michael Wright [Wed, 18 Sep 2019 17:29:04 +0000 (19:29 +0200)]
Work around qpath_res issue
Josh Mcguigan [Sun, 15 Sep 2019 17:33:20 +0000 (10:33 -0700)]
remove machine applicable suggestion explicit_write format #4542
mikerite [Wed, 18 Sep 2019 04:48:47 +0000 (06:48 +0200)]
Add note about refactoring
Co-Authored-By: Philipp Krones <hello@philkrones.com>
flip1995 [Tue, 17 Sep 2019 07:48:25 +0000 (09:48 +0200)]
Add ICE regression test
Jason Olson [Sun, 15 Sep 2019 18:07:44 +0000 (11:07 -0700)]
Changes to catch_fatal_errors in rustc driver
A [recent PR](https://github.com/rust-lang/rust/pull/60584/files#diff-707a0eda6b2f1a0537abc3d23133748cL1151)
changed the function name from `report_ices_to_stderr_if_any` to `catch_fatal_errors`. This PR changes to using
the new function name.
Jason Olson [Fri, 13 Sep 2019 00:30:58 +0000 (17:30 -0700)]
Changes cast-lossless to a pedantic lint
Fixes #4528
bors [Sat, 14 Sep 2019 16:02:59 +0000 (16:02 +0000)]
Auto merge of #4513 - matthiaskrgr:dogdood_incrcomp, r=llogiq
tests: disable incremental compilation in dogfood to reduce target dir size
tests: execute dogfood tests with incremental compilation disabled reduces target/ dir size of when "cargo test"ing by around 2 gigs.
changelog: none
bors [Sat, 14 Sep 2019 06:12:52 +0000 (06:12 +0000)]
Auto merge of #4540 - jolson88:fix-place-projection, r=phansch
Fix rustc breaking change: convert to Place's new boxed slice projection
Addressed breaking changes from rust-lang PR https://github.com/rust-lang/rust/pull/63420/
I'm not entirely sure the semantics are preserved as I don't have much knowledge about MIR yet. So this code was largely reverse-engineered from the PR above. I wouldn't be surprised if I did something wrong :).
I followed the instructions to pull latest rustc from master and verified the build break I was seeing in my PR for cast-lossless in Travis CI. With these changes, it compiles again and all tests pass.
Fixes https://github.com/rust-lang/rust/issues/64440
changelog: none
Jason Olson [Sat, 14 Sep 2019 01:49:11 +0000 (18:49 -0700)]
Convert to Place's new boxed slice projection
Fixes breaking change from rust-lang PR https://github.com/rust-lang/rust/pull/63420/
Michael Wright [Fri, 13 Sep 2019 06:20:27 +0000 (08:20 +0200)]
Apply review suggestions
Fix grammar errors and use `Pat::is_rest` instead of own function.
bors [Thu, 12 Sep 2019 08:10:43 +0000 (08:10 +0000)]
Auto merge of #4536 - matthiaskrgr:impl_lint_doc, r=flip1995
docs: fix broken link in adding_lints documantation
changelog: none
bors [Thu, 12 Sep 2019 07:42:38 +0000 (07:42 +0000)]
Auto merge of #4538 - mikerite:fix-3567, r=flip1995
Verify that issue #3567 is resolved
Closes #3567
changelog: none
Michael Wright [Thu, 12 Sep 2019 06:47:11 +0000 (08:47 +0200)]
Verify that issue #3567 is resolved
Closes #3567
Michael Wright [Thu, 12 Sep 2019 06:36:05 +0000 (08:36 +0200)]
Fix lint warnings
Michael Wright [Thu, 12 Sep 2019 06:25:05 +0000 (08:25 +0200)]
Add `unneeded-wildcard-pattern` lint
Matthias Krüger [Wed, 11 Sep 2019 19:53:12 +0000 (21:53 +0200)]
docs: fix broken link in adding_lints documantation
Matthias Krüger [Wed, 11 Sep 2019 18:52:36 +0000 (20:52 +0200)]
tests: dogfood: run tests with debuginfo=0 reducing dogfood target dir size even further.
Matthias Krüger [Fri, 6 Sep 2019 17:35:20 +0000 (19:35 +0200)]
tests: execute dogfood tests with incremental compilation disabled
reduces target/ dir size of when "cago test"ing by around 2 gigs.
bors [Wed, 11 Sep 2019 09:48:25 +0000 (09:48 +0000)]
Auto merge of #4478 - tsurai:master, r=flip1995
Fix incorrect swap suggestion
Clippy suggests using swap on fields belonging to the same owner causing two mutable borrows of the owner.
Disclosure: This is my first time working with clippy and rusts HIR. I'm very grateful for assistance and suggestions to improve the PR.
fixes #981
changelog: Fix false positive in `manual_swap` lint
bors [Wed, 11 Sep 2019 07:14:31 +0000 (07:14 +0000)]
Auto merge of #4534 - matthiaskrgr:fmt_, r=flip1995
run rustfmt
I was getting local `cargo test` failures because of this, yet travis was green.
changelog: none
Matthias Krüger [Wed, 11 Sep 2019 06:26:57 +0000 (08:26 +0200)]
run rustfmt