| Age | Commit message (Collapse) | Author | Lines | |
|---|---|---|---|---|
| 2019-12-22 | Fix `map_clone` false positive | Michael Wright | -0/+10 | |
| Don't lint when the item type is not a reference. `copied` only applies to references. | ||||
| 2019-10-03 | Test fixes | Shotaro Yamada | -1/+1 | |
| 2019-05-16 | Also rename the lint in test files | flip1995 | -1/+1 | |
| 2019-05-16 | Split redundant_closure lint | Michael Wright | -1/+1 | |
| Move the method checking into a new lint called `redundant_closures_for_method_calls` and put it in the pedantic group. This aspect of the lint seems more controversial than the rest. cc #3942 | ||||
| 2019-04-15 | Only suggest .copied() for Option right now | Manish Goregaokar | -2/+2 | |
| 2019-04-15 | Also suggest .copied() when .clone() is called on a Copy type | Manish Goregaokar | -0/+1 | |
| 2019-04-15 | Suggest .copied() instead of .cloned() in map_clone when dealing with references | Manish Goregaokar | -0/+1 | |
| 2019-02-10 | removing redundant closures in the whole project | Grzegorz | -0/+1 | |
| 2019-01-15 | Really fix issue number in `map_clone` test | Michael Wright | -1/+1 | |
| 2019-01-15 | Fix `map_clone` bad suggestion | Michael Wright | -0/+12 | |
| `cloned` requires that the elements of the iterator must be references. This change determines if that is the case by examining the type of the closure argument and suggesting `.cloned` only if it is a reference. When the closure argument is not a reference, it suggests removing the `map` call instead. A minor problem with this change is that the new check sometimes overlaps with the `clone_on_copy` lint. Fixes #498 | ||||
| 2019-01-13 | Add run-rustfix to map_clone test | Wilco Kusee | -0/+2 | |
| 2019-01-08 | Remove all copyright license headers | Philipp Hansch | -9/+0 | |
| Discussion previously happened in https://github.com/rust-lang/rust/pull/43498 | ||||
| 2018-12-09 | rustfmt tests | Matthias Krüger | -2/+0 | |
| 2018-10-11 | Stabilize tool lints | Oliver Scherer | -1/+1 | |
| 2018-10-08 | Don't suggest cloned() for map Box deref | Ryan Cumming | -0/+1 | |
| Boxes are a bit magic in that they need to use `*` to get an owned value out of the box. They implement `Deref` but that only returns a reference. This means an easy way to convert an `Option<Box<T>>` to an `<Option<T>` is: ``` box_option.map(|b| *b) ``` However, since b36bb0a6 the `map_clone` lint is detecting this as an attempt to copy the box. Fix by excluding boxes completely from the deref part of this lint. Fixes #3274 | ||||
| 2018-10-06 | Add license header to Rust files | Manish Goregaokar | -0/+10 | |
| 2018-10-02 | Reimplement the `map_clone` lint from scratch | Oliver Schneider | -0/+9 | |
| 2018-10-02 | relicensing: Remove map_clone | Manish Goregaokar | -105/+0 | |
| This removes the code added in https://github.com/rust-lang-nursery/rust-clippy/pull/427 | ||||
| 2018-08-29 | Adapt ui-tests to the tool_lints | flip1995 | -3/+3 | |
| 2017-10-03 | Reduce the hackiness of cargo-clippy | Oliver Schneider | -2/+2 | |
| 2017-08-01 | Move all our tests back to ui tests | Oliver Schneider | -0/+105 | |
| 2017-05-17 | Make most tests an `example` so we can run them one by one | Oliver Schneider | -105/+0 | |
| 2017-05-11 | Address PR comments | Oliver Schneider | -3/+5 | |
| 2017-05-10 | Fixes #443 | Oliver Schneider | -1/+4 | |
| 2017-02-08 | remove all //~ from tests | Oliver Schneider | -22/+22 | |
| 2017-02-07 | rustup and compile-fail -> ui test move | Oliver Schneider | -0/+100 | |
