R.Chavignat
|
ab481e5cb1
|
Refactored the CastPass lints.
|
2015-08-20 21:37:37 +02:00 |
|
R.Chavignat
|
ff28dd324e
|
Fixed a little oversight.
|
2015-08-20 14:50:26 +02:00 |
|
R.Chavignat
|
14528d433a
|
Simplified reexported ast::* type paths.
Also removed trailing whitespaces.
|
2015-08-20 14:46:40 +02:00 |
|
R.Chavignat
|
b417f01ed8
|
Also test that the CastExpr's right arm is numeric.
|
2015-08-20 14:36:26 +02:00 |
|
R.Chavignat
|
93d9249f76
|
Moved allow(unknown_lints) to crate level.
|
2015-08-20 14:25:08 +02:00 |
|
R.Chavignat
|
1846581baa
|
Added examples to lint descriptions.
|
2015-08-20 14:24:26 +02:00 |
|
R.Chavignat
|
993239d33a
|
Initial implementation of lossy cast lints.
Introduces 3 lints :
cast_possible_overflow
cast_precision_loss
cast_sign_loss
Add a compile-test test case.
Fix errors spotted by dogfood script.
|
2015-08-20 01:04:06 +02:00 |
|
llogiq
|
150840667e
|
Merge pull request #204 from Manishearth/bit-mask
Extend and correct the ineffective_bit_mask check
|
2015-08-19 13:25:43 +02:00 |
|
llogiq
|
973d5e5c6b
|
Mul and Div for integers
|
2015-08-19 11:58:59 +02:00 |
|
llogiq
|
98d24b5b56
|
fixed #203 and #197
|
2015-08-19 09:39:03 +02:00 |
|
Georg Brandl
|
8f4499f3ae
|
new lint: comparing unit types (fixes #201)
|
2015-08-19 08:11:00 +02:00 |
|
llogiq
|
9f67ba7f8d
|
re-applied birkenfeld's improvements
|
2015-08-18 14:18:36 +02:00 |
|
llogiq
|
6e0c103133
|
more small const improvements
|
2015-08-18 12:26:01 +02:00 |
|
Georg Brandl
|
49ad73f6e4
|
consts: minor improvements
|
2015-08-18 11:07:13 +02:00 |
|
Georg Brandl
|
7f52239cab
|
consts: convert to using a struct with state
Struct has the context reference (as an Option) and the
needed_resolution flag.
|
2015-08-17 19:58:21 +02:00 |
|
Georg Brandl
|
a2dcbfea65
|
const eval: implement ! for integers
|
2015-08-17 16:45:56 +02:00 |
|
llogiq
|
a2ee637be6
|
added test and fixed negativity check in Partial{Eq, Ord} impl
|
2015-08-17 16:24:57 +02:00 |
|
llogiq
|
c47402416b
|
Added bit operations to const folding
|
2015-08-17 15:11:36 +02:00 |
|
llogiq
|
4394362836
|
dogfooding
|
2015-08-17 13:23:17 +02:00 |
|
llogiq
|
9f134f8e95
|
added PartialEq/PartialOrd to ConstantVariant, used to implement comparing binops
|
2015-08-17 13:18:14 +02:00 |
|
llogiq
|
0e67c0134f
|
make float_cmp check for zero
|
2015-08-17 12:08:45 +02:00 |
|
llogiq
|
fb715ce45d
|
fix 189, fixed a few warnings, ==/!= for consts, refactored consts test
|
2015-08-17 12:08:45 +02:00 |
|
Manish Goregaokar
|
4f1fcd4d5b
|
Merge pull request #184 from Manishearth/identity_op
Identity op now uses const folding, no longer follows const bindings
|
2015-08-17 14:35:45 +05:30 |
|
Manish Goregaokar
|
65deb956f1
|
Merge pull request #186 from birkenfeld/fix
loops: fix two trailing periods in lint msgs
|
2015-08-17 11:05:02 +05:30 |
|
Georg Brandl
|
caeb72c47b
|
loops: fix two trailing periods in lint msgs
|
2015-08-17 07:30:33 +02:00 |
|
Georg Brandl
|
e9a41e2374
|
new lint: lint when iterating over any Iterator::next() result (fixes #182)
|
2015-08-17 07:28:40 +02:00 |
|
llogiq
|
759b45a46d
|
made is_negative(..) public (+doctest), fixed identity_op and precedence
|
2015-08-16 23:09:56 +02:00 |
|
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 |
|