Daniel Marjamäki
|
7847263451
|
Fixed #6158 (New check: dangerous sign conversion)
|
2014-09-11 18:10:19 +02:00 |
PKEuS
|
865fc9aa67
|
Fixed #6154: Don't suggest to reduce scope if inner scope is a lambda.
|
2014-09-10 20:56:34 +02:00 |
Daniel Marjamäki
|
a46f5c00a8
|
Minor tweaks for new CheckOther::checkIntegerOverflow checker
|
2014-09-10 17:26:20 +02:00 |
Daniel Marjamäki
|
974c8688c3
|
Fixed #1751 (Undefined Behavior: Signed integer overflow)
|
2014-09-10 17:02:18 +02:00 |
Daniel Marjamäki
|
a36b0e55be
|
CheckOther::checkTooBigBitwiseShift: improving error messages
|
2014-09-09 17:06:45 +02:00 |
Daniel Marjamäki
|
f111a89639
|
Fixed #1757 (Undefined Behavior: Shift too many bits)
|
2014-09-09 07:24:59 +02:00 |
Martin Ettl
|
0ced0c33e5
|
Fixed #6149: (gcc) warning: comparing floating point with == or != is unsafe. Using Tokenizer::isOneNumber function to compare a floating point number being 1.0.
|
2014-09-08 20:02:04 +02:00 |
PKEuS
|
301b668eb5
|
Fixed internal warning of cppcheck
|
2014-09-07 11:47:40 +02:00 |
PKEuS
|
ff4fc6a234
|
New check: Recommend expm1, log1p, erfc (#5392)
|
2014-09-07 11:38:49 +02:00 |
Alexander Mai
|
efab840b50
|
#5230 Explicit reinterpret_cast should not give a warning. Lower all invalidPointerCast messages to 'portability'
|
2014-09-06 13:09:02 +02:00 |
Alexander Mai
|
29b46cb505
|
#6132 crash: daca: kvirc CheckOther::checkRedundantAssignment()
|
2014-09-05 20:03:34 +02:00 |
PKEuS
|
e8f7279039
|
Refactorization: Moved detection of STL strings to SymbolDatabase
|
2014-09-05 12:03:08 +02:00 |
PKEuS
|
800b57d87e
|
Merge pull request #415 from Dmitry-Me/reducePatternsDuplication
Reduce patterns duplication
|
2014-09-04 07:46:59 +02:00 |
Alexander Mai
|
45dc4ca6be
|
Fix compiler warning. Re-order if-expression for performance reasons
|
2014-09-02 19:18:30 +02:00 |
Dmitry-Me
|
46d24c7bfb
|
Reduce patterns duplication
|
2014-09-02 19:51:22 +04:00 |
PKEuS
|
ddc19febb5
|
Merge pull request #413 from Dmitry-Me/improveCStyleCastsDetection
Improve C style casts detection
|
2014-09-02 08:07:53 +02:00 |
PKEuS
|
428f7a6f26
|
Attempt to fix crashs on clang test suite.
|
2014-09-01 23:29:15 +02:00 |
PKEuS
|
00044aabb0
|
Handle ternary operator in redundantAssignment check (#5964)
|
2014-09-01 23:16:52 +02:00 |
Alexander Mai
|
a4ff30301a
|
#5805 'Passing value -1.0 to sqrt() leads to undefined result' is incorrect. Degrade wrongmathcall from error to warning, since it deals with implementation-defined behaviour
|
2014-09-01 19:31:32 +02:00 |
Dmitry-Me
|
f937dde1e0
|
Improve C style casts detection
|
2014-09-01 16:40:28 +04:00 |
PKEuS
|
c92230dfee
|
Fixed crash on garbage code introduced by recent commit.
|
2014-09-01 12:41:30 +02:00 |
PKEuS
|
20a066bb14
|
Run unreachableCode checking only on executable scopes (#5789)
|
2014-09-01 11:29:39 +02:00 |
PKEuS
|
b5e064e737
|
Fixed unreachableCode message if a lambda is returned (#6008).
|
2014-09-01 11:10:42 +02:00 |
PKEuS
|
353a9e9a64
|
Bailout for duplicateBranch check if branches are empty (#5354)
|
2014-09-01 10:19:14 +02:00 |
PKEuS
|
e9fdfc6cd0
|
Fix on checking side for #6099
|
2014-08-30 11:18:48 +02:00 |
PKEuS
|
06a92e8981
|
Moved several condition checks from checkOther to checkCondition (former checkAssignIf)
|
2014-08-29 17:06:46 +02:00 |
PKEuS
|
eba8c6f6c5
|
Refactorization:
- Added missing separating comments between checks in checkother.cpp
- Moved checks related to strings into own file
|
2014-08-27 09:42:09 +02:00 |
PKEuS
|
7f2be2f57c
|
Fixed template bracket linkage in while loop simplification
Ran AStyle
|
2014-08-23 12:28:54 +02:00 |
amai2012
|
976966fe81
|
#5639 String literal compared with char buffer in a struct.
|
2014-08-23 09:41:40 +02:00 |
PKEuS
|
0bf7b03f96
|
Refactorization: Support :: in some places in checkother.cpp
|
2014-08-18 10:25:30 +02:00 |
Daniel Marjamäki
|
d44d6ad94a
|
incorrect logic operator: fixed crashes
|
2014-08-18 05:37:07 +02:00 |
Daniel Marjamäki
|
ebc0b6cd44
|
astIsFloat: Fix crash when there is no second operand for '.'
|
2014-08-17 14:28:31 +02:00 |
Daniel Marjamäki
|
f7f44f24c7
|
Fixed #5132 (False negative: incorrectLogicOperator in simple if-clause)
|
2014-08-17 14:04:40 +02:00 |
Daniel Marjamäki
|
364c975701
|
Fixed #5557 (astIsFloat: better handling of '.')
|
2014-08-15 16:48:53 +02:00 |
PKEuS
|
5d302716e7
|
Refactorized Variable::isIntegralType() and Variable::isFloatType():
- Cached property
- Make use of it in several checks
- float* is flagged as floating point type
|
2014-08-09 11:45:13 +02:00 |
PKEuS
|
c3577bc8dd
|
Fixed crash, simplified code in testassert.cpp
|
2014-08-06 12:24:21 +02:00 |
PKEuS
|
d3a567bf96
|
Fixed false positive #5824: Variables that might be used for va_start() must not be passed by reference.
|
2014-08-06 12:06:36 +02:00 |
PKEuS
|
4a3d719386
|
Fixed recently introduced FP when C-Style casting nullpointers
|
2014-08-04 11:42:14 +02:00 |
PKEuS
|
4c80ca4e8e
|
Improved cstyleCast check:
- Detect casting of literals
- Check initializer lists (#3630)
|
2014-08-04 11:21:25 +02:00 |
Daniel Marjamäki
|
8a1c5d6fce
|
Fixed #6011 (crash: libreoffice/binfilter svt_enhwmf.cxx)
|
2014-07-28 16:33:42 +02:00 |
Daniel Marjamäki
|
fdfea717c6
|
Suspicious string comparison: Refactoring using AST. Fixed FP in Lac.
|
2014-07-28 14:27:35 +02:00 |
Daniel Marjamäki
|
76020d2ad0
|
astIsFloat: expression 'floatvar ? 0x40 : 0' is not float
|
2014-07-26 09:09:58 +02:00 |
Dmitry-Me
|
07c120f1af
|
Cache option flags and check them first.
|
2014-07-23 17:06:27 +04:00 |
Daniel Marjamäki
|
2cc31acabb
|
Fixed #5922 (FP: passedByValue for std::string::value_type)
|
2014-07-18 15:51:59 +02:00 |
Daniel Marjamäki
|
bf5b4d9ece
|
astyle formatting
[ci skip]
|
2014-07-14 14:38:40 +02:00 |
Dmitry-Me
|
9829e0b48a
|
Clearer variable names, fix typo in comment.
|
2014-07-11 11:33:09 +04:00 |
Dmitry-Me
|
ba3757dfd3
|
Bring variables decalaration to where they are really needed.
|
2014-07-10 11:23:14 +04:00 |
Daniel Marjamäki
|
d40b77dce2
|
Removed special 'else if' handling. this is redundant since these are simplified.
|
2014-07-02 16:16:19 +02:00 |
Daniel Marjamäki
|
5e9e90b1c1
|
Fixed Token::Match pattern. In x|y|.. patterns, all %cmd% should be placed before plain-text operands.
|
2014-07-01 15:55:29 +02:00 |
Daniel Marjamäki
|
41baffdda1
|
Fixed #5850 (Wrong Message on self assignment)
|
2014-06-28 15:26:22 +02:00 |