rail
e855fe3620
Reflect the changes that has been made and fmt
2020-09-25 09:02:05 +12:00
rail
f410df3c48
make clippy happy (needless_pass_by_value
, filter_map
and find_map
)
2020-09-25 09:02:05 +12:00
rail
44187383f4
Use operator overloading instead of direct calls of make_binop
2020-09-25 09:02:05 +12:00
rail
d9a88be0b0
Rename get_offset
and its private items
2020-09-25 09:02:05 +12:00
rail
8da6cfd17b
fmt
2020-09-25 09:02:05 +12:00
rail
de56279cd9
Implement building the manual_memcpy
sugggestion with loop counters
2020-09-25 09:02:05 +12:00
rail
720f19f2ec
Implement detecting manual_memcpy
with loop counters
2020-09-25 09:02:05 +12:00
rail
b4b4da162f
Introduce Start and StartKind
2020-09-25 09:02:05 +12:00
rail
1026b42f06
Rename a struct and variables
2020-09-25 09:02:05 +12:00
rail
9573a0d378
Rename variables
2020-09-25 09:02:05 +12:00
rail
13c207d375
Generalise InitializeVisitor
2020-09-25 09:02:05 +12:00
rail
c599e2fcfa
Split VarState
2020-09-25 09:02:05 +12:00
rail
31cb110964
add concinient methods to Increment/InitializeVisitor
2020-09-25 08:59:14 +12:00
rail
b2d5b89a1d
Check if it's after the loop earlier
2020-09-25 08:59:14 +12:00
rail
116f30dc33
Use else blocks instead of return statements in Increment/InitializeVisitor
2020-09-25 08:59:14 +12:00
rail
dc89bb1135
Use if_chain in Increment/InitializeVisitor
2020-09-25 08:59:14 +12:00
rail
5e393c7747
Fix a FP in explicit_counter_loop
...
Fix a false positive in `explicit_counter_loop` where the loop counter is used after incremented,
adjust the test so that counters are incremented at the end of the loop
and add the test for this false positive.
2020-09-23 08:15:44 +12:00
Michael Wright
8b04c2d6e8
Merge branch 'master' into lint-5734
2020-09-15 21:21:35 +02:00
Eduardo Broto
332c2dcb4d
Fix dogfood after MatchTypeOnDiagItem
2020-09-15 10:29:53 +02:00
Michael Wright
15244a88df
Fix manual-strip
dogfood errors
2020-09-14 06:11:35 +02:00
Takayuki Nakata
1d8ae3aa12
Address items_after_statement
2020-09-08 23:12:04 +09:00
Takayuki Nakata
5d085ad011
Some refactoring
2020-09-08 08:34:51 +09:00
Takayuki Nakata
619ca76731
Refactoring: use inner function
2020-09-08 08:25:31 +09:00
Takayuki Nakata
3d30ef7818
Restrict same_item_push
to suppress false positives
...
It emits a lint when the pushed item is a literal, a constant and an immutable binding that are initialized with those.
2020-09-07 23:46:43 +09:00
bors
e9440cbcde
Auto merge of #5997 - giraffate:fix_fp_about_clone_in_same_item_push, r=ebroto
...
Fix FP in `same_item_push`
Don't emit a lint when the pushed item doesn't have Clone trait
Fix #5979
changelog: Fix FP in `same_item_push` not to emit a lint when the pushed item doesn't have Clone trait
2020-09-05 20:27:17 +00:00
Takayuki Nakata
96b31a5b36
Fix FP when coercion kicks in
2020-09-06 00:02:35 +09:00
LeSeulArtichaut
28f9b84042
ty.kind
-> ty.kind()
in rustdoc and clippy
2020-09-04 18:27:33 +02:00
Takayuki Nakata
aa7ffa5257
Fix FP in same_item_push
...
Don't emit a lint when the pushed item doesn't have Clone trait
2020-09-01 22:39:09 +09:00
flip1995
282c59820b
Merge commit '3d0b0e66afdfaa519d8855b338b35b4605775945' into clippyup
2020-08-28 18:43:25 +02:00
flip1995
c680602005
Merge remote-tracking branch 'upstream/master' into rustup
2020-08-18 19:50:23 +02:00
David Wood
f13d2bfd9b
clippy: support QPath::LangItem
...
This commit updates clippy with the introduction of `QPath::LangItem` so
that it still compiles.
Signed-off-by: David Wood <david@davidtw.co>
2020-08-17 13:55:05 +01:00
Takayuki Nakata
99ba290a14
Improve code style
2020-08-17 08:36:02 +09:00
Takayuki Nakata
f98ffa271d
Fix FP for same_item_push
...
Don't emit a lint when `pushed_item` was declared as mutable variable.
2020-08-14 22:54:12 +09:00
flip1995
027780ca2c
Merge commit '09bd400243ed6f7059fedc0c1623aae3792521d6' into clippyup
2020-08-11 17:50:45 +02:00
Philipp Krones
9da5b6d1d0
Rollup merge of #5825 - giraffate:same_item_push, r=Manishearth
...
Add the new lint `same_item_push`
changelog: Add the new lint `same_item_push`
Fixed #4078 . As I said in https://github.com/rust-lang/rust-clippy/issues/4078#issuecomment-658184195 , I referrerd to https://github.com/rust-lang/rust-clippy/pull/4647 .
2020-08-10 14:56:25 +02:00
Eduardo Broto
888657e09a
Fix ICE in loops
module
2020-08-08 18:13:43 +02:00
Takayuki Nakata
610d4e3c8b
rustfmt
2020-08-05 23:10:30 +09:00
Takayuki Nakata
e48685edef
Just check if it contains _
in for pat
2020-08-05 23:00:03 +09:00
Takayuki Nakata
228f668282
Use mutated_variables
2020-08-05 23:00:03 +09:00
Takayuki Nakata
b7ceb4d3d7
rustfmt
2020-08-05 23:00:03 +09:00
Takayuki Nakata
14a4e3bcc8
Fix a lint message
2020-08-05 22:59:12 +09:00
Takayuki Nakata
2beb9090d1
Rename TypeckTables to TypeckResults
2020-08-05 22:51:38 +09:00
Takayuki Nakata
161f475100
Add test case for same_item_push
2020-08-05 22:51:38 +09:00
Takayuki Nakata
1e8ada3cab
Add lint same_item_push
2020-08-05 22:51:38 +09:00
Philipp Krones
fb7ad956f6
Rollup merge of #5856 - phansch:remove-symbol-reexport, r=flip1995
...
Remove old Symbol reexport
I couldn't really tell what it was meant to improve. It seems more clear
without the renaming to `Name`?
changelog: none
2020-08-04 12:06:43 +02:00
JarredAllen
5e10b039a3
Implement review suggestions
2020-08-02 21:46:18 -07:00
JarredAllen
bb2c14e92b
Fix a bug causing it to be too trigger-happy
2020-08-02 21:34:17 -07:00
JarredAllen
a849483294
Fix formatting and dogfood fallout
2020-08-02 21:34:17 -07:00
JarredAllen
3657c92ac9
Check for other things which can be used indirectly
2020-08-02 21:34:17 -07:00
JarredAllen
3ee61373fe
Write the lint and write tests
2020-08-02 21:34:17 -07:00