]>
git.lizzy.rs Git - rust.git/log
Tim Bodeit [Sat, 23 Nov 2019 21:35:21 +0000 (22:35 +0100)]
[comparison_chain] #4827 Check `core::cmp::Ord` is implemented
Only emit lint, if `cmp` is actually available on the type being
compared. Don't emit lint in cases where only `PartialOrd` is
implemented.
bors [Sat, 23 Nov 2019 17:40:57 +0000 (17:40 +0000)]
Auto merge of #4840 - flip1995:rollup-jqk3a3i, r=flip1995
Rollup of 5 pull requests
Successful merges:
- #4730 (Fix check_infinite_loop (while_immutable_condition) by checking for break or return inside loop body)
- #4766 (Fix false positive in derive_hash_xor_eq)
- #4811 (Literal Representation Restructure)
- #4820 (doc: fix the comment above the lint function)
- #4830 (use more efficient code to generate repeated string)
Failed merges:
r? @ghost
changelog: none
Philipp Krones [Sat, 23 Nov 2019 17:16:03 +0000 (18:16 +0100)]
Rollup merge of #4830 - lzutao:str-repeat, r=flip1995
use more efficient code to generate repeated string
see https://rust.godbolt.org/z/z9vrFP for comparison
changelog: none
Philipp Krones [Sat, 23 Nov 2019 17:16:01 +0000 (18:16 +0100)]
Rollup merge of #4820 - guanqun:comment-fix, r=flip1995
doc: fix the comment above the lint function
it's a simple comment fix.
---
changelog: none
Philipp Krones [Sat, 23 Nov 2019 17:16:00 +0000 (18:16 +0100)]
Rollup merge of #4811 - mikerite:lit_repr_20191113, r=flip1995
Literal Representation Restructure
This pull request restructures the literal_representation module to be easier to understand and maintain. I split the changes into a lot of commits to make reviewing easier.
changelog: none
Philipp Krones [Sat, 23 Nov 2019 17:15:59 +0000 (18:15 +0100)]
Rollup merge of #4766 - phansch:fix_fp_in_derive_hash_xor_eq, r=flip1995
Fix false positive in derive_hash_xor_eq
This fixes a false positive in derive_hash_xor_eq where the lint was
triggering on user-defined traits called `Hash`.
changelog: Fix false positive in `derive_hash_xor_eq`
Fixes #4658
Philipp Krones [Sat, 23 Nov 2019 17:15:58 +0000 (18:15 +0100)]
Rollup merge of #4730 - yerke:fix-check_infinite_loop, r=flip1995
Fix check_infinite_loop (while_immutable_condition) by checking for break or return inside loop body
changelog: Fix check_infinite_loop (while_immutable_condition) by checking for break or return inside loop body
fixes #4648
bors [Sat, 23 Nov 2019 17:15:11 +0000 (17:15 +0000)]
Auto merge of #4839 - flip1995:rollup-new-lints, r=flip1995
Rollup of 4 Pull requests with new lints
Rollup of pull requests
- #4816 (New lint: zst_offset)
- #4814 (New lint: Implement ifs_same_cond_fn)
- #4807 (Add `large_stack_arrays` lint)
- #4806 (Issue/4623)
changelog: add [`zst_offset`] lint
changelog: New lint: [`ifs_same_cond_fn`]
cahngelog: Add new lint [large_stack_arrays]
changelog: added lint [`tabs_in_doc_comments`]
flip1995 [Sat, 23 Nov 2019 17:09:09 +0000 (18:09 +0100)]
Fix fallout
flip1995 [Sat, 23 Nov 2019 16:57:28 +0000 (17:57 +0100)]
Run update_lints
flip1995 [Sat, 23 Nov 2019 16:56:13 +0000 (17:56 +0100)]
Merge remote-tracking branch 'FlorianRohm/issue/4623' into rollup-new-lints
flip1995 [Sat, 23 Nov 2019 16:55:10 +0000 (17:55 +0100)]
Merge remote-tracking branch 'Areredify/large_stack_arrays' into rollup-new-lints
flip1995 [Sat, 23 Nov 2019 16:54:26 +0000 (17:54 +0100)]
Merge remote-tracking branch 'popzxc/if-same-cond-fn' into rollup-new-lints
flip1995 [Sat, 23 Nov 2019 16:53:41 +0000 (17:53 +0100)]
Merge remote-tracking branch 'upstream/zst-offset' into rollup-new-lints
bors [Sat, 23 Nov 2019 15:54:33 +0000 (15:54 +0000)]
Auto merge of #4825 - Manishearth:rustup, r=flip1995
Rustup to rustc 1.41.0-nightly (
35ef33a8 2019-11-21)
I don't have the right fix for the fmtstr tests, and I'm also hitting problems caused by https://github.com/messense/rustc-test/issues/3
List of rustups:
- rust-lang/rust#66271 (syntax: Keep string literals in ABIs and `asm!` more precisely)
- rust-lang/rust#65355 (Stabilize `!` in Rust 1.41.0)
- rust-lang/rust#66515 (Reduce size of `hir::Expr` by boxing more of `hir::InlineAsm`)
- rust-lang/rust#66389 (Specific labels when referring to "expected" and "found" types)
- rust-lang/rust#66074 ([mir-opt] Turn on the `ConstProp` pass by default)
changelog: none
flip1995 [Sat, 23 Nov 2019 15:49:47 +0000 (16:49 +0100)]
Disable chalk integration test. Output too large
flip1995 [Sat, 23 Nov 2019 15:47:59 +0000 (16:47 +0100)]
Temporarily disable serde integration test
ICEs because of rustc, not Clippy
flip1995 [Sat, 23 Nov 2019 00:26:19 +0000 (01:26 +0100)]
Add comment for mir_opt_level=0
Yerkebulan Tulibergenov [Fri, 25 Oct 2019 06:46:25 +0000 (23:46 -0700)]
fix check_infinite_loop by checking for break or return inside loop body
flip1995 [Fri, 22 Nov 2019 13:24:38 +0000 (14:24 +0100)]
Set mir_opt_level=0
This introduces some FNs. But a building Clippy is more important for now
flip1995 [Fri, 22 Nov 2019 12:46:20 +0000 (13:46 +0100)]
Rustup to rust-lang/rust#66389
flip1995 [Fri, 22 Nov 2019 12:25:38 +0000 (13:25 +0100)]
Rustup to rust-lang/rust#66515
Fixes #4835
flip1995 [Fri, 22 Nov 2019 12:24:17 +0000 (13:24 +0100)]
Remove never_type feature
Stablized in rust-lang/rust#65355
Igor Aleksanov [Thu, 14 Nov 2019 05:06:34 +0000 (08:06 +0300)]
Implement if_same_cond_fn lint
Run ./util/dev
Revert changelog entry
Rename lint to same_functions_in_if_condition and add a doc example
Add testcases with different arg in fn invocation
lzutao [Tue, 19 Nov 2019 16:47:18 +0000 (23:47 +0700)]
use usize::from
Co-Authored-By: Mateusz MikuĊa <mati865@users.noreply.github.com>
Lzu Tao [Tue, 19 Nov 2019 16:08:50 +0000 (23:08 +0700)]
use more efficient code to generate repeated string
see https://rust.godbolt.org/z/z9vrFP for comparison
Manish Goregaokar [Tue, 19 Nov 2019 08:32:35 +0000 (00:32 -0800)]
Rustup to rustc 1.41.0-nightly (
d1da8023d 2019-11-19)
Manish Goregaokar [Mon, 18 Nov 2019 15:38:36 +0000 (07:38 -0800)]
Rustup to rustc 1.41.0-nightly (
a0d40f8bd 2019-11-18)
Andre Bogus [Thu, 14 Nov 2019 19:18:24 +0000 (20:18 +0100)]
New lint: zst_offset
Guanqun Lu [Fri, 15 Nov 2019 17:06:57 +0000 (01:06 +0800)]
doc: fix the comment above the lint function
Florian Rohm [Fri, 15 Nov 2019 15:20:21 +0000 (16:20 +0100)]
register new lint "tabs in doc comments" and update readme
Florian Rohm [Fri, 15 Nov 2019 15:18:08 +0000 (16:18 +0100)]
add new lint tabs in doc comments
bors [Thu, 14 Nov 2019 10:00:18 +0000 (10:00 +0000)]
Auto merge of #4815 - JohnTitor:rustup, r=matthiaskrgr
Rustup rust-lang/rust#66233
changelog: none
Yuki Okushi [Thu, 14 Nov 2019 08:13:07 +0000 (17:13 +0900)]
Rustup rust-lang/rust#66233
Michael Wright [Thu, 14 Nov 2019 06:08:24 +0000 (08:08 +0200)]
literal repr: ignore more warnings in macros
Michael Wright [Thu, 14 Nov 2019 05:42:04 +0000 (07:42 +0200)]
literal representation: simplification
Simplify calculation in grouping. Add test case to ensure `count()`
can't be zero in that branch.
Areredify [Tue, 12 Nov 2019 22:22:53 +0000 (22:22 +0000)]
Add new lint: large stack array
added documentation
minor style fix
change as to ::from
add ignore to doc
include threshold in lint message/make suggestion more apparent/use Scalar api instead of matching
style fix
shange snippet_opt to snippet
bors [Wed, 13 Nov 2019 18:07:23 +0000 (18:07 +0000)]
Auto merge of #4810 - Areredify:4716-fix, r=flip1995
clippy-driver display help on empty command line arguments
changelog: fixes #4716, now displaying help if clippy-driver is run with no arguments.
bors [Wed, 13 Nov 2019 13:13:14 +0000 (13:13 +0000)]
Auto merge of #4794 - rust-lang:no-must-use-impls, r=flip1995
no more must-use-candidate impls
This should help with #4779.
changelog: don't trigger [`must_use_candidate`] on trait impls
Michael Wright [Wed, 13 Nov 2019 06:28:50 +0000 (08:28 +0200)]
literal representation restructure 12
Export function for formatting literals and remove crate visibility from
other items.
Michael Wright [Wed, 13 Nov 2019 06:28:06 +0000 (08:28 +0200)]
literal representation restructure 11
Rename `grouping_hint` to `format` and use the term consistently.
Michael Wright [Wed, 13 Nov 2019 06:28:01 +0000 (08:28 +0200)]
literal representation restructure 10
Rename DigitInfo to NumericLiteral
Michael Wright [Wed, 13 Nov 2019 06:27:54 +0000 (08:27 +0200)]
literal representation restructure 9
Only store valid suffixes (and not mistyped suffixes) in DigitInfo.
Check for mistyped suffixes later and not when DigitInfo is created.
This opens the door to more sophisticated mistyped suffix checks later.
Michael Wright [Wed, 13 Nov 2019 06:27:49 +0000 (08:27 +0200)]
literal representation restructure 8
Store the digit parts directly in DigitInfo since we need them anyway.
Michael Wright [Wed, 13 Nov 2019 06:27:42 +0000 (08:27 +0200)]
literal representation restructure 7
Replace `do_lint` with `get_group_size`. Return `None` if there are no
groups.
Michael Wright [Wed, 13 Nov 2019 06:27:37 +0000 (08:27 +0200)]
literal representation restructure 6
Add `group_digits` helper function.
Michael Wright [Wed, 13 Nov 2019 06:27:27 +0000 (08:27 +0200)]
literal representation restructure 5
Use `split_digit_parts` in `check_lit`.
Michael Wright [Wed, 13 Nov 2019 06:27:19 +0000 (08:27 +0200)]
literal representation restructure 4
Simplify `grouping_hint` by splitting digits into parts and handling
one at a time.
Fixes #4762
Michael Wright [Wed, 13 Nov 2019 06:27:14 +0000 (08:27 +0200)]
literal representation restructure 3
Move suffix check into `check_lit` so that it isn't done repeatedly.
Michael Wright [Wed, 13 Nov 2019 06:27:05 +0000 (08:27 +0200)]
literal representation restructure 2
Consolidate warning handling using "poor man's try".
Michael Wright [Wed, 13 Nov 2019 06:26:52 +0000 (08:26 +0200)]
literal representation restructure 1
Combine macro expansion checks. Indentation is a little strange to
avoid rustfmt issue.
Mikhail Babenko [Wed, 13 Nov 2019 01:27:43 +0000 (04:27 +0300)]
display help on empty command line arguments
Andre Bogus [Tue, 12 Nov 2019 22:36:22 +0000 (23:36 +0100)]
no more must-use-candidate on trait impls
bors [Tue, 12 Nov 2019 13:56:14 +0000 (13:56 +0000)]
Auto merge of #4769 - euclio:crlf, r=flip1995
don't warn on CRLF in `with_newline` lints
changelog: don't warn on CRLF in `print_with_newline` and `write_with_newline`
fixes #4208.
This PR also transitions the unescaping logic to use the compiler's lexer.
Andy Russell [Sun, 3 Nov 2019 04:41:22 +0000 (00:41 -0400)]
don't warn on CRLF in `with_newline` lints
bors [Mon, 11 Nov 2019 19:24:20 +0000 (19:24 +0000)]
Auto merge of #4803 - tomprogrammer:issue-4732, r=phansch
Fix false positive in explicit_counter_loop lint
When the counter was used in a closure after the loop the lint didn't detect the
usage of the counter correctly.
changelog: Fix false positive in `explicit_counter_loop`
Fixes #4732
bors [Mon, 11 Nov 2019 18:59:06 +0000 (18:59 +0000)]
Auto merge of #4805 - Manishearth:rustup, r=phansch
Rustup to rustc 1.40.0-nightly (
56237d75b 2019-11-11)
changelog: none
Manish Goregaokar [Mon, 11 Nov 2019 17:24:12 +0000 (09:24 -0800)]
MutImmutable -> Immutable, MutMutable -> Mutable, CaptureClause -> CaptureBy
bors [Mon, 11 Nov 2019 11:28:27 +0000 (11:28 +0000)]
Auto merge of #4801 - mikerite:to_digit_is_some, r=flip1995
To digit is some
Add a lint that recommends replacing `to_digit().is_some()` with `is_digit()` on `char`s
changelog: Add lint `to_digit_is_some`
Thomas Bahn [Mon, 11 Nov 2019 10:27:18 +0000 (11:27 +0100)]
Fix false positive in explicit_counter_loop lint
When the counter was used in a closure after the loop the lint didn't detect the
usage of the counter correctly.
Philipp Hansch [Fri, 1 Nov 2019 19:12:08 +0000 (20:12 +0100)]
Fix false positive in derive_hash_xor_eq
This fixes a false positive in derive_hash_xor_eq where the lint was
triggering on user-defined traits called `Hash`.
bors [Mon, 11 Nov 2019 06:23:27 +0000 (06:23 +0000)]
Auto merge of #4780 - flip1995:ice_4775, r=phansch
Fix ICE #4775
Fixes #4775
changelog: Fix ICE with const_generics
bors [Mon, 11 Nov 2019 06:03:56 +0000 (06:03 +0000)]
Auto merge of #4802 - mikerite:fix-breakage-
20191111 , r=phansch
Fix breakage due to rust-lang/rust#65324
changelog: none
Michael Wright [Mon, 11 Nov 2019 04:22:50 +0000 (06:22 +0200)]
Fix breakage due to rust-lang/rust#65324
Michael Wright [Sun, 10 Nov 2019 16:00:04 +0000 (18:00 +0200)]
Really fix `to_digit_is_some` documentation
Michael Wright [Sun, 10 Nov 2019 15:55:25 +0000 (17:55 +0200)]
Fix `to_digit_is_some` documentation
Michael Wright [Sun, 10 Nov 2019 13:14:26 +0000 (15:14 +0200)]
Fix `to_digit_is_some` dogfood errors
Michael Wright [Sun, 10 Nov 2019 13:12:05 +0000 (15:12 +0200)]
Add `to_digit_is_some` lint
bors [Sun, 10 Nov 2019 04:51:23 +0000 (04:51 +0000)]
Auto merge of #4800 - msizanoen1:target-libs2, r=Manishearth
Only pass `--extern` flags for rlibs in target deps directory
It seems like there are proc macros/build scripts using serde: https://github.com/rust-lang/rust/pull/66207#issuecomment-
552159692
This fix the build error by only passing `--extern` flags for rlibs in target deps directory (if available, otherwise host deps directory).
r? @Manishearth
changelog: none
msizanoen1 [Sun, 10 Nov 2019 03:54:49 +0000 (10:54 +0700)]
Only pass rlibs in target deps directory
bors [Sun, 10 Nov 2019 01:10:12 +0000 (01:10 +0000)]
Auto merge of #4799 - Manishearth:rustup, r=yaahc
Rustup to rustc 1.40.0-nightly (
ac162c6ab 2019-11-09)
changelog: none
Manish Goregaokar [Sun, 10 Nov 2019 01:04:42 +0000 (17:04 -0800)]
Rustup to rustc 1.40.0-nightly (
ac162c6ab 2019-11-09)
bors [Fri, 8 Nov 2019 23:33:30 +0000 (23:33 +0000)]
Auto merge of #4791 - lzutao:osx2bors, r=matthiaskrgr
build macOS on bors branches only
Clippy is portable enough. It is rare to see Clippy builds' broke
on *nix OSes. Testing macOS build on auto and try branch is
enough.
changelog: none
bors [Fri, 8 Nov 2019 22:58:33 +0000 (22:58 +0000)]
Auto merge of #4786 - msizanoen1:target-libs, r=Manishearth
Add the TARGET_LIBS environment variable for rustc CI testing
Needed to fix the test failure in rust-lang/rust#66158.
See https://github.com/rust-lang/rust/pull/66158#issuecomment-
550585396
r? @Manishearth
changelog: none
bors [Fri, 8 Nov 2019 22:06:21 +0000 (22:06 +0000)]
Auto merge of #4776 - mikerite:fix-4727, r=flip1995
Fix crash in `use-self` lint
Fixes #4727
changelog: Fix crash in `use-self` lint
bors [Fri, 8 Nov 2019 21:27:26 +0000 (21:27 +0000)]
Auto merge of #4795 - HMPerson1:rustup, r=matthiaskrgr
Rustup rust-lang/rust#66188
changelog: none
HMPerson1 [Fri, 8 Nov 2019 20:41:54 +0000 (15:41 -0500)]
dogfood
HMPerson1 [Fri, 8 Nov 2019 20:12:08 +0000 (15:12 -0500)]
Rustup rust-lang/rust#66188
msizanoen1 [Fri, 8 Nov 2019 05:29:07 +0000 (12:29 +0700)]
Add target libs directory to search path
Michael Wright [Fri, 8 Nov 2019 05:18:12 +0000 (07:18 +0200)]
Put ice test auxiliary fix in right place
Michael Wright [Fri, 8 Nov 2019 05:15:16 +0000 (07:15 +0200)]
Merge branch 'master' into fix-4727
bors [Fri, 8 Nov 2019 03:34:42 +0000 (03:34 +0000)]
Auto merge of #4792 - lzutao:rustup-65916, r=Manishearth
rustup rust-lang/rust#65916
cc rust-lang/rust#65916
changelog: none
Lzu Tao [Fri, 8 Nov 2019 03:16:43 +0000 (10:16 +0700)]
remove unused warnings
Lzu Tao [Fri, 8 Nov 2019 03:10:08 +0000 (10:10 +0700)]
rustup https://github.com/rust-lang/rust/pull/65916
bors [Thu, 7 Nov 2019 22:05:51 +0000 (22:05 +0000)]
Auto merge of #4697 - Licenser:no-exit, r=flip1995
restriction lint for `std::process::exit`
Addition to #4655 - adds the lint checking for `std::process::exit`
changelog: add restriction lint for `std::process::exit`
bors [Thu, 7 Nov 2019 21:33:04 +0000 (21:33 +0000)]
Auto merge of #4784 - Manishearth:remove-extern, r=Manishearth
Remove `extern crate clippy_lints` from tests
This causes rustc's build system to fail because it still tries to load the crate as a plugin: https://github.com/rust-lang/rust/pull/66158#issuecomment-
550415160 . I'm not sure _why_ this happens, but for a short term fix we should remove these.
In one case it was just a convenient crate to use so i picked a different test. In another it was load-bearing, I had to delete the test. Idk if there's a better way around this.
changelog: none
bors [Thu, 7 Nov 2019 21:06:18 +0000 (21:06 +0000)]
Auto merge of #4758 - mikerite:dec_lit_20191031, r=flip1995
`DecimalLiteralRepresentation` simplification
Remove recalculation of literal value.
changelog: none
Manish Goregaokar [Thu, 7 Nov 2019 20:48:22 +0000 (12:48 -0800)]
Remove clippy dependency in lint_without_lint_pass
Manish Goregaokar [Wed, 6 Nov 2019 18:36:04 +0000 (10:36 -0800)]
Remove clippy_lints from useless attribute test
bors [Thu, 7 Nov 2019 20:47:14 +0000 (20:47 +0000)]
Auto merge of #4757 - evanjs:issue/4748, r=phansch
Fix Deprecated lints don't expand
### Move doc comments inside of declare_deprecated_lint macros so that they are picked up by lintlib.py
### fixes #4748
Unable to `cargo test` locally (I'm on NixOS, and keep getting errors that are similar to those #4714 might solve) but I have verified that all deprecated lints can now be expanded like other lints.
![2019-10-30_21:06:28](https://user-images.githubusercontent.com/
1847524 /
67910501 -
5815de00 -fb59-11e9-9fa2-
91fe6a8b9bb9 .png)
changelog: Show deprecated lints in lint documentation again
Lzu Tao [Thu, 7 Nov 2019 19:06:28 +0000 (02:06 +0700)]
build macOS on bors branches only
Clippy is portable enough. It is rare to see Clippy builds' broke
on *nix OSes. Testing macOS build on auto and try branch is
enough.
Heinz N. Gies [Thu, 7 Nov 2019 16:13:26 +0000 (17:13 +0100)]
Update lints
Heinz N. Gies [Tue, 29 Oct 2019 20:34:00 +0000 (21:34 +0100)]
Update tests for exit
Heinz N. Gies [Fri, 25 Oct 2019 08:59:33 +0000 (10:59 +0200)]
Update clippy_lints/src/lib.rs
Co-Authored-By: Philipp Krones <hello@philkrones.com>
Heinz N. Gies [Fri, 25 Oct 2019 08:16:25 +0000 (10:16 +0200)]
Update clippy_lints/src/lib.rs
Co-Authored-By: Philipp Krones <hello@philkrones.com>
Heinz N. Gies [Thu, 24 Oct 2019 19:17:21 +0000 (21:17 +0200)]
Simplify dentry point detection
Heinz N. Gies [Mon, 21 Oct 2019 20:55:01 +0000 (22:55 +0200)]
Improve function checking
Heinz N. Gies [Sat, 19 Oct 2019 12:31:02 +0000 (14:31 +0200)]
Exclude main from exit lint
Heinz N. Gies [Fri, 18 Oct 2019 19:10:35 +0000 (21:10 +0200)]
Update lints
Heinz N. Gies [Fri, 18 Oct 2019 19:09:42 +0000 (21:09 +0200)]
Add lint for exit