llogiq
|
c4b07ead17
|
Merge pull request #179 from nweston/step-by-zero
New lint: Range::step_by(0) (fixes #95)
|
2015-08-16 20:12:52 +02:00 |
|
Nathan Weston
|
23a38c4170
|
New lint: Range::step_by(0) (fixes #95)
Uses type information so it can detect non-literal ranges as well
(Range or RangeFrom -- the other range types don't have step_by).
|
2015-08-16 12:58:54 -04:00 |
|
llogiq
|
4244f2479f
|
dogfooding, part 2
|
2015-08-16 16:09:00 +02:00 |
|
llogiq
|
fe0de07b28
|
dogfooded
|
2015-08-16 16:05:51 +02:00 |
|
llogiq
|
03c7d7074d
|
With working test now
|
2015-08-16 15:56:09 +02:00 |
|
Manish Goregaokar
|
164907ece2
|
restrict toplevel_ref_arg to only functions (fixes #170)
|
2015-08-16 17:24:03 +05:30 |
|
Georg Brandl
|
64954283c1
|
add some imports to guard against crate moves
|
2015-08-16 09:03:06 +02:00 |
|
Georg Brandl
|
47b605304d
|
all: organize imports
* remove unused imports
* separate external and internal imports
* consistent import of rustc::lint
* move #[allow(unused_imports)] to local impl
|
2015-08-16 08:55:34 +02:00 |
|
Georg Brandl
|
997f345046
|
all: remove unneeded as_str() calls for ast::Name
Name has PartialEq and Display impls that do what we want.
|
2015-08-16 08:38:47 +02:00 |
|
Manish Goregaokar
|
f494f14aa6
|
Merge pull request #177 from birkenfeld/if_let_mini_fix
misc: fix check for unit body in "match -> if let" lint (fixes #172)
|
2015-08-15 14:12:34 +05:30 |
|
Georg Brandl
|
542bf8d50e
|
misc: fix check for unit body in "match -> if let" lint (fixes #172)
|
2015-08-15 09:56:15 +02:00 |
|
Georg Brandl
|
b299433de3
|
lifetimes: fix case with one unnamed and one static ref (fixes #171)
|
2015-08-15 09:36:07 +02:00 |
|
llogiq
|
b1920cb21c
|
Merge pull request #174 from birkenfeld/small_fixes
methods: allow Option.unwrap by default
|
2015-08-15 08:27:33 +02:00 |
|
Georg Brandl
|
137a9f3def
|
methods: allow Option.unwrap by default
|
2015-08-15 07:23:04 +02:00 |
|
llogiq
|
f23af0cfd5
|
changed const to consts to avoid keyword, added test, fixed a lot of bugs
|
2015-08-14 17:14:54 +02:00 |
|
llogiq
|
b6ac44d5b2
|
Removed #![allow(redundant_closure)] on eq_op
|
2015-08-14 15:00:04 +02:00 |
|
llogiq
|
5d99ebec72
|
fixed false positives (at the cost of some false negatives)
|
2015-08-14 14:37:12 +02:00 |
|
Manish Goregaokar
|
763ae1f3ae
|
Fix dogfood
|
2015-08-13 23:20:00 +05:30 |
|
Manish Goregaokar
|
5ce8e7ba85
|
trim_multiline: ignore empty lines
|
2015-08-13 23:18:34 +05:30 |
|
Manish Goregaokar
|
fbbb44d93b
|
Handle tabs
|
2015-08-13 23:18:34 +05:30 |
|
Manish Goregaokar
|
83487c060f
|
Add trim_multiline utility (fixes #139)
|
2015-08-13 23:18:33 +05:30 |
|
Manish Goregaokar
|
847070e19d
|
Merge pull request #151 from birkenfeld/needless_bool_msg
spelling fix, rework needless_bool with snippet (fixes #150)
|
2015-08-13 23:17:18 +05:30 |
|
Manish Goregaokar
|
3cf5c36296
|
Address review comments, move to travis
|
2015-08-13 21:45:50 +05:30 |
|
Georg Brandl
|
49e51fe65a
|
lifetimes: try to fix w.r.t. lifetimes from parent scopes (fixes #162)
|
2015-08-13 21:45:42 +05:30 |
|
Manish Goregaokar
|
09db7f3fee
|
fix
|
2015-08-13 21:45:42 +05:30 |
|
Manish Goregaokar
|
c2bdc85715
|
oh the irony
|
2015-08-13 21:45:41 +05:30 |
|
Manish Goregaokar
|
5eab397e7c
|
Some fixes from dogfooding clippy
|
2015-08-13 21:45:41 +05:30 |
|
Georg Brandl
|
9578403638
|
new lint: looping over x.iter() or x.iter_mut() (fixes #157)
|
2015-08-13 16:31:16 +02:00 |
|
Georg Brandl
|
8a98736f51
|
spelling fix, rework needless_bool with snippet (fixes #150)
|
2015-08-13 15:31:16 +02:00 |
|
Georg Brandl
|
e03b71606b
|
update_lints: add a check mode for travis runs
|
2015-08-13 15:30:06 +02:00 |
|
Manish Goregaokar
|
1d64553bc6
|
Merge branch 'pr-152'
Conflicts:
README.md
src/lib.rs
src/strings.rs
|
2015-08-13 18:38:08 +05:30 |
|
llogiq
|
6aa36e9deb
|
initial addition and subtraction for bytes and ints
|
2015-08-13 14:22:05 +02:00 |
|
llogiq
|
0ba08715e5
|
Merge
|
2015-08-13 11:41:50 +02:00 |
|
Andre Bogus
|
a00270c5b1
|
grammar
|
2015-08-13 11:38:39 +02:00 |
|
llogiq
|
9e786d3956
|
added string_add to clippy lint group
|
2015-08-13 11:36:39 +02:00 |
|
llogiq
|
f9e851e212
|
pulled strings passes together, added more tests
|
2015-08-13 11:36:39 +02:00 |
|
llogiq
|
52c0cf5a9d
|
fixed formatting
|
2015-08-13 11:36:39 +02:00 |
|
llogiq
|
0b08e9e83e
|
added string_add lint and fixed string_add_assign + test
|
2015-08-13 11:36:39 +02:00 |
|
llogiq
|
71b46d9ecd
|
improved string_add/string_add_assign messages, Allow-by-default string_add_assign
|
2015-08-13 11:35:30 +02:00 |
|
Georg Brandl
|
ea0cf2a296
|
update script: also generate lint list in lib.rs
|
2015-08-13 11:34:51 +02:00 |
|
Georg Brandl
|
2c2716f045
|
all: DRY for lint descriptions
* use the rustc style for lint descriptions
* add a script to parse all lint descriptions
and put the generated table into README
|
2015-08-13 11:14:05 +02:00 |
|
llogiq
|
1a19d5ef65
|
changed Constant to a struct with 'needed_resolution' bool
|
2015-08-13 10:48:12 +02:00 |
|
llogiq
|
a2f19f2a38
|
added follow flag
|
2015-08-13 10:48:12 +02:00 |
|
llogiq
|
6aeb955214
|
fixed if-condition match
|
2015-08-13 10:48:12 +02:00 |
|
llogiq
|
8b9c2a79ed
|
First (incomplete) const folding
|
2015-08-13 10:48:11 +02:00 |
|
Manish Goregaokar
|
8dfa02938d
|
Merge branch 'pr-149'
|
2015-08-13 13:00:29 +05:30 |
|
Manish Goregaokar
|
510ea0c5e5
|
Merge branch 'pr-140'
Conflicts:
src/lib.rs
|
2015-08-13 12:52:14 +05:30 |
|
Georg Brandl
|
a67e55f3f0
|
lint messages: remove trailing period
Since lint messages often are suffixed by ", #[warn(xxx)] on by default"
this trailing period produces an ugly clash with the comma.
|
2015-08-13 08:15:42 +02:00 |
|
Georg Brandl
|
7aee04878f
|
tests: use fragment of lint text for error checking
(Did not touch strings.rs, which is fixed by @llogiq's PR)
|
2015-08-13 08:12:07 +02:00 |
|
Georg Brandl
|
2f7693094f
|
lifetimes lint: include support for lifetimes as generic params
|
2015-08-13 06:43:25 +02:00 |
|