Skip to main content
summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2016-08-17Update version to 9.1.0Marc Khouzam1-1/+1
This commit used the script releng/scripts/ChangeFeaturesVersion.sh Change-Id: I8274a6e8dd5c0f1e450f07952e138b265a441a62
2016-07-06Bug 496628 - Avoid codan markers that cover an entire class declarationNathan Ridge1-0/+15
This fixes a regression from bug 486610 which introduced these in some cases. Change-Id: I791528ce7f0bc061386aaa97dd9cecb7abeecd72
2016-06-22Move the rest of the CDT plugins to java 8Marc Khouzam3-5/+5
This change was generated using the script: releng/scripts/ChangeJavaVersion.sh Change-Id: I2ad96dc682a5acb8529c3edec40de279c331b5a4
2016-04-16Bug 491825 - Remove primitive wrapper creationAlex Blewitt1-2/+2
Using `new Integer` and other wrapper types such as `new Character` results in potential extra heap utilisation as the values are not cached. The built-in `Integer.valueOf` will perform caching on numbers in the range -128..127 (at least) using a flyweight pattern. In addition, parsing `int` values can be done with `Integer.parseInt` which avoids object construction. Adjust tests such as `"true".equals(expr)` to `Boolean.parseBoolean(expr)`. Change-Id: I0408a5c69afc4ca6ede71acaf6cc4abd67538006 Signed-off-by: Alex Blewitt <alex.blewitt@gmail.com>
2016-02-23Bug 486610 - Fixed a highlight problem in reportProblem methodAlisson Linhares de Carvalho1-0/+15
Every time we report a problem in a declaration/expression that occupies more than one line, the codan marks the entire line as a problem - including everything before the node. This patch solves this inconvenient behavior. See https://bugs.eclipse.org/bugs/show_bug.cgi?id=486610 for a complete explanation. Change-Id: I742cbaac8a1392676695d574355597b0cfc87385 Signed-off-by: Alisson Linhares de Carvalho <arescarv@gmail.com>
2016-02-22Code cleanup in Codan tests.Sergey Prigogin25-377/+383
Change-Id: Ib6fbc348ef175441029a17dd2129d95aaee3305f
2016-02-11Update copyright datesMarc Khouzam2-2/+2
Change-Id: I94ac38c37657e270f62db9bbd2859c2c6de40e5f
2016-01-18Incremented CDT features version to 9.0.0Marc Khouzam1-1/+1
This includes bumping the standalone debugger to 9.0.0 Change-Id: I55fae1cec995b5f6865a5b174c004ed1b76e9bea
2016-01-11Bug 383576 - Ability to ignore codan errors using line commentsAlena Laskavaia1-0/+40
Change-Id: I806e1787fb6cc3be8865cee1d397d581ae4acd8e Signed-off-by: Alena Laskavaia <elaskavaia.cdt@gmail.com>
2016-01-03Fix copyright of all CDT plugins using the copyright tool from platform.Marc Khouzam23-24/+24
This commit does not add missing copyrights, just updates the date on the existing ones. Change-Id: I646f5afd533a1fcc539bdf2e0686b22f406ecf65
2015-12-23Bug 484619 - Make sure ProblemBindingChecker reports unresolvedNathan Ridge1-0/+14
user-defined literal operators Change-Id: Idbb157c33b3aae06a00a10f7e7836e96db9f9b63 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com>
2015-11-29Bug 479638 - Cache the result of EvalBinding.isConstantExpression() Nathan Ridge1-1/+11
This helps avoid infinite recursion when a variable's initializer references itself. Change-Id: I4667536ebbefd2008afe9003617092a0a5693db0 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com>
2015-09-29Remove reference to java environment in build.propertiesMarc Khouzam1-2/+1
Change-Id: I053158a23d2783879248320a84a9549e28cfb4d1
2015-06-10Incremented CDT feature version to 8.8.0Marc Khouzam1-1/+1
Change-Id: Ib1189a08a5f0225fd676b682dc8a10477ad3acc5 Signed-off-by: Marc Khouzam <marc.khouzam@ericsson.com>
2015-05-04Incremented CDT feature version to 8.7.0Marc Khouzam1-1/+1
Change-Id: Ife0feaaa9263d2b7797e31d628250bd26caff1b5 Signed-off-by: Marc Khouzam <marc.khouzam@ericsson.com>
2015-04-06Bug 461538 - "noreturn" attribute in destructor is not accounted forSergey Prigogin1-7/+17
Change-Id: Ia3c3bdfdb26ff18092fd20be02f178ebb71848bb Signed-off-by: Sergey Prigogin <eclipse.sprigogin@gmail.com>
2015-03-03converted codan test to use comment labels positioningAlena Laskavaia1-27/+14
Change-Id: Ib389c90a8b10abac46399248f435428e27b6975b
2015-02-25codan - checker for commentsAlena Laskavaia5-16/+162
implement few code style rules for comments - no line comments (sometimes enforced) - no nested comments (no /* in block comment) Change-Id: Ib4042a8ffab0dc1c5effd7b77ab6d2f51a1e6cae
2015-02-22codan - code cleanupAlena Laskavaia3-2/+5
Change-Id: I25f1047b9b07388adeeab38ba7679ac3109280fb
2015-02-21codan tests - removed sleeps and made sure it runs in non-ui threadAlena Laskavaia1-12/+23
Change-Id: Ic684e647ea43968c7da35e0f9448e01e66c06e8f
2015-02-21codan tests - make use of comment tags in testsAlena Laskavaia3-89/+68
Change-Id: I822ea36bfa49164b43954ae80bc9fd223ae8d0f6
2015-02-20Bug 455828 - Don't be over-eager when collecting dead nodes in theNathan Ridge2-3/+28
control flow graph Change-Id: I54013e31a197c02698e3161f9f52755e4cb6b203 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com>
2015-02-18Bug 455828 - Proper handling of 'switch' without explicit defaultNathan Ridge2-4/+46
(including empty switch) Change-Id: I3e20400f86c5e4273d8b0c62ed9ac3f429a84879 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com>
2015-02-18Bug 455828 - Proper handling of 'break' inside compound statement in aNathan Ridge1-0/+21
case of a switch Change-Id: I73329a8769bc20a4fc4e017faad627e03adce9d9 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com>
2015-02-18Bug 327375 control flow graph regression - dead connector nodesAlena Laskavaia2-24/+105
basically for the code like if (a) return 1; else return 2; a++; // this should be a dead code node but we don't check connector node of "if" and don't add it to dead nodes graph resulting it this code not being marked as dead, which can lead to false positives or false negatives for checkers that use that. Same apply for other control statements... Change-Id: Iafb4b48ca960f0ffab2c0265285a31cb08eb0d11
2015-02-16Bug 441714 - Consider spurious semicolons in CaseBreakCheckerNathan Ridge1-0/+14
Change-Id: Id4fe394164063007c45da37ae82cc730a9e726dd Signed-off-by: Nathan Ridge <zeratul976@hotmail.com>
2015-02-06Extract test VM args to common property base.test.vmargsMarc-Andre Laperle1-1/+1
This makes it easier to change arguments for all tests. As a side-effect, all tests now enable assertion (-ea) and have the same max heap size, etc. Change-Id: I0965d40ba3283efe7eacdf927f6c302ee0ab30f0 Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
2015-01-06Incremented CDT feature version to 8.6.0.Sergey Prigogin1-1/+1
Change-Id: Ic90c8e6a68d21e5b90ec3fac997da6a95a29bedf Signed-off-by: Sergey Prigogin <eclipse.sprigogin@gmail.com> Reviewed-on: https://git.eclipse.org/r/39066 Tested-by: Hudson CI
2014-12-14Bug 447486 - codan - CCE is thrown by control flow graph builderAlena Laskavaia1-0/+42
- addOutgoing cannot join JumpNode - block will be added to dead list instead - addjusted goto code to not use addOutgoing but using setJump directly - corrected code that doing fake return to deal jump nodes at the end - corrected code that marks up dead code to remove jump targets - corrected code that calculates all nodes in the graph to take into account dangling labels - fixed CFG viewer to show unconnected labeled statements Change-Id: Ie4d9e37678e3ebaae8e9f268e6f37342e14a1444 Reviewed-on: https://git.eclipse.org/r/38189 Tested-by: Hudson CI Reviewed-by: Elena Laskavaia <elaskavaia.cdt@gmail.com>
2014-12-08Bug 452325 - Protect against IVariable.getInitialValue() calling itselfNathan Ridge1-0/+9
recursively along any code path Change-Id: Idbf46ca82b431e72bf6b2598427430c37ef65cc4 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com> Reviewed-on: https://git.eclipse.org/r/37220 Tested-by: Hudson CI Reviewed-by: Elena Laskavaia <elaskavaia.cdt@gmail.com> Reviewed-by: Sergey Prigogin <eclipse.sprigogin@gmail.com> Tested-by: Sergey Prigogin <eclipse.sprigogin@gmail.com>
2014-12-04Bug 453275 - Codan tests cannot test for checkers throwing an exceptionAlena Laskavaia1-0/+2
Added system flag that is used to propagate exception instead of swallowing. Set by default in codan junit tests. Change-Id: Iccda9a7b2e419192bb9f4eec4b89d525e152fc81 Signed-off-by: Alena Laskavaia <elaskavaia.cdt@gmail.com> Reviewed-on: https://git.eclipse.org/r/37482
2014-09-12Bug 353352 - CFG doesn't handle range-based for loopAlena Laskavaia1-1/+2
Change-Id: Ic4334ce82585d4dbfbbd008fd013b0ce97cca434 Reviewed-on: https://git.eclipse.org/r/33155 Tested-by: Hudson CI Reviewed-by: Elena Laskavaia <elaskavaia.cdt@gmail.com>
2014-09-09Bug 353352 - CFG doesn't handle range-based for loopAlena Laskavaia1-6/+63
Change-Id: Ic71c15d26d31d954fbc4c91882dd9361d9b23eae
2014-08-28Add missing arg for jacoco in in tests. Update jacoco.Marc-Andre Laperle1-1/+1
Jacoco arguments are missing when executing certain tests. This happens when argLine is being overriden. We had the same problem in Linux Tools. We solved it by prepending the argLine with ${tycho.testArgLine}. This should help improving the Code coverage metric in Sonar. Also update the jacoco version to fix a bug when building with Java 8. Change-Id: I7f73f40488efc01fc7d643940c1ac0d5950c900d Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com> Reviewed-on: https://git.eclipse.org/r/32507 Tested-by: Hudson CI Reviewed-by: Alvaro Sanchez-Leon <alvsan09@gmail.com>
2014-07-21Inremented branding plugin version to 8.5Sergey Prigogin1-1/+1
2014-06-21Bug 394521 Do Constant folding propagation when building CFGAlena Laskavaia2-2/+66
When building control flow graph we can consider constants and not add some branches in decisions nodes, that make CFG more accurate and elimate some false positives Change-Id: I19c623e1e6d2dc7416a0e796a98168df7a976111 Signed-off-by: Alena Laskavaia <elaskavaia.cdt@gmail.com> Reviewed-on: https://git.eclipse.org/r/28726
2014-06-19Bug 433357 - Added fix for false positives from GNUChris Marr1-1/+10
Statement-Expressions Change-Id: I21eb9978a156f2375e5b9e1992c297e3b6ca78ac Signed-off-by: Chris Marr <cdabone@gmail.com> Reviewed-on: https://git.eclipse.org/r/25600 Reviewed-by: Sergey Prigogin <eclipse.sprigogin@gmail.com> Tested-by: Hudson CI Reviewed-by: Elena Laskavaia <elaskavaia.cdt@gmail.com>
2014-06-16Bug 350168 - Return checker won't report errors in dead code Alena Laskavaia1-0/+38
Also Bug 356908, Bug 348386 Change-Id: I48d2f74e05d2d6d7a7bf0589408ca90bc07a6922 Reviewed-on: https://git.eclipse.org/r/28527 Tested-by: Hudson CI Reviewed-by: Doug Schaefer <dschaefer@qnx.com>
2014-06-03Build with 4.4 and use target definition file for maven buildMarc-Andre Laperle1-0/+6
Change-Id: I1044751c600e7725fd2cb02578c28d5f45de1854 Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com> Reviewed-on: https://git.eclipse.org/r/27631 Tested-by: Hudson CI Reviewed-by: Doug Schaefer <dschaefer@qnx.com>
2014-04-06Bug 430282 - Limit recursion depth when processing inheritanceNathan Ridge1-0/+9
hierarchies Change-Id: Ia9d9f379cb61e8d8bc8d1fb33f893fcd2a0688a2 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com> Reviewed-on: https://git.eclipse.org/r/23957 Reviewed-by: Sergey Prigogin <eclipse.sprigogin@gmail.com> IP-Clean: Sergey Prigogin <eclipse.sprigogin@gmail.com> Tested-by: Sergey Prigogin <eclipse.sprigogin@gmail.com>
2014-03-15Fix intermittent test failures in codanMarc-Andre Laperle1-0/+2
Sometimes, index-based checkers (AbstractIndexAstChecker) were skipped because the indexer was not fully initialized so expected errors were not getting reported. Change-Id: Ibb764224f11ae4a867a4aa6b9de2b8ec55e90c55 Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com> Reviewed-on: https://git.eclipse.org/r/23428 Reviewed-by: Nathan Ridge <zeratul976@hotmail.com> Tested-by: Hudson CI Reviewed-by: Sergey Prigogin <eclipse.sprigogin@gmail.com>
2014-01-24Enabled JDK 1.7.Sergey Prigogin4-6/+6
Change-Id: Ic2a161ea3c318dc1e8ba01c271c936bf5ed8763d Reviewed-on: https://git.eclipse.org/r/21020 Reviewed-by: Sergey Prigogin <eclipse.sprigogin@gmail.com> IP-Clean: Sergey Prigogin <eclipse.sprigogin@gmail.com> Tested-by: Sergey Prigogin <eclipse.sprigogin@gmail.com>
2014-01-21Bumped up CDT version to 8.4.Sergey Prigogin1-1/+1
Change-Id: I1229344feaaed4a3551ceb7b1ef1410545048b55 Reviewed-on: https://git.eclipse.org/r/20908 Reviewed-by: Sergey Prigogin <eclipse.sprigogin@gmail.com> IP-Clean: Sergey Prigogin <eclipse.sprigogin@gmail.com> Tested-by: Sergey Prigogin <eclipse.sprigogin@gmail.com>
2013-12-16Bug 419938 - [fp] Pure virtual implementation not recognized if only oneNathan Ridge1-0/+18
path implements it Change-Id: I90732a87d8d6b4ad2a84aa3c8b09b10727afa994 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com> Reviewed-on: https://git.eclipse.org/r/19823 Tested-by: Hudson CI Reviewed-by: Sergey Prigogin <eclipse.sprigogin@gmail.com> IP-Clean: Sergey Prigogin <eclipse.sprigogin@gmail.com> Tested-by: Sergey Prigogin <eclipse.sprigogin@gmail.com>
2013-09-05Bug 416284 - Unsafe method call, using nested templatesNathan Ridge1-0/+16
Change-Id: I08732af62f43c03f362d4602736ef6b8e0f227e9 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com> Reviewed-on: https://git.eclipse.org/r/16147 Reviewed-by: Sergey Prigogin <eclipse.sprigogin@gmail.com> IP-Clean: Sergey Prigogin <eclipse.sprigogin@gmail.com> Tested-by: Sergey Prigogin <eclipse.sprigogin@gmail.com>
2013-08-27Fixed parent pom version to referenceAndrew Gvozdev1-1/+1
2013-08-18Bug 414279 - Derived class method that overrides a base class methodNathan Ridge1-0/+19
brought into the derived scope with a using-declaration Change-Id: Ibeb49dddb7ef3768ffd4964e22a139afba8690fb Signed-off-by: Nathan Ridge <zeratul976@hotmail.com> Reviewed-on: https://git.eclipse.org/r/15573 Reviewed-by: Sergey Prigogin <eclipse.sprigogin@gmail.com> IP-Clean: Sergey Prigogin <eclipse.sprigogin@gmail.com> Tested-by: Sergey Prigogin <eclipse.sprigogin@gmail.com>
2013-07-28Bug 351612 - Pure virtual implementation not recognized if multiplyNathan Ridge1-0/+54
inherited Change-Id: I1e9141fbb5cc72bb7b59b77d6faf958726094e5f Signed-off-by: Nathan Ridge <zeratul976@hotmail.com> Reviewed-on: https://git.eclipse.org/r/14906 Reviewed-by: Sergey Prigogin <eclipse.sprigogin@gmail.com> IP-Clean: Sergey Prigogin <eclipse.sprigogin@gmail.com> Tested-by: Sergey Prigogin <eclipse.sprigogin@gmail.com>
2013-07-24Bug 352267 - Assignment in condition checker matches the wholeNathan Ridge2-0/+35
expression Change-Id: I7aea1c5d404a8d374020e34c997e6e890fcdb3d6 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com> Reviewed-on: https://git.eclipse.org/r/14764 Reviewed-by: Sergey Prigogin <eclipse.sprigogin@gmail.com> IP-Clean: Sergey Prigogin <eclipse.sprigogin@gmail.com> Tested-by: Sergey Prigogin <eclipse.sprigogin@gmail.com>
2013-07-19Bug 358694 - False warning about Unused static function in templateNathan Ridge1-0/+16
Change-Id: I0ebeef9db921239e896983c68e5291316209c2d2 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com> Reviewed-on: https://git.eclipse.org/r/14680 Reviewed-by: Sergey Prigogin <eclipse.sprigogin@gmail.com> IP-Clean: Sergey Prigogin <eclipse.sprigogin@gmail.com> Tested-by: Sergey Prigogin <eclipse.sprigogin@gmail.com>

Back to the top