Skip to main content
aboutsummaryrefslogtreecommitdiffstats
path: root/codan
AgeCommit message (Collapse)AuthorFilesLines
2015-02-22codan tests - removed sleeps and made sure it runs in non-ui threadAlena Laskavaia1-12/+23
Change-Id: Ic684e647ea43968c7da35e0f9448e01e66c06e8f
2015-02-22codan 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 Ridge4-18/+67
control flow graph Change-Id: I54013e31a197c02698e3161f9f52755e4cb6b203 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com>
2015-02-19Bug 455828 - Proper handling of 'switch' without explicit defaultNathan Ridge3-4/+58
(including empty switch) Change-Id: I3e20400f86c5e4273d8b0c62ed9ac3f429a84879 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com>
2015-02-19Bug 455828 - Proper handling of 'break' inside compound statement in aNathan Ridge2-22/+45
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 Laskavaia3-35/+135
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-18Bug 455828 - General cleanup in ControlFlowGraphBuilderNathan Ridge1-18/+11
Change-Id: I46cfe23fc10bed235d091afaec8eec29dd7d62ac Signed-off-by: Nathan Ridge <zeratul976@hotmail.com>
2015-02-16Bug 441714 - Consider spurious semicolons in CaseBreakCheckerNathan Ridge2-0/+25
Change-Id: Id4fe394164063007c45da37ae82cc730a9e726dd Signed-off-by: Nathan Ridge <zeratul976@hotmail.com>
2015-02-16Fix a typo in org.eclipse.cdt.codan.ui.cfgview/plugin.xmlNathan Ridge1-1/+1
Change-Id: I41aeb1195e9d8b663de396883e0aa6e4f3b5c6e4 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com>
2015-02-11Remove unnecessary .api_filtersMarc Khouzam1-12/+0
Note that memory/org.eclipse.cdt.debug.ui.memory.traditional/.settings/.api_filters needs to be kept because: 1- it removes warnings about implementing provisional APIs 2- that plugin sets warnings as errors, so warnings are not allowed Change-Id: Id4bcc9cd78c4c6678ef749cfe276e5a9945efa1e Signed-off-by: Marc Khouzam <marc.khouzam@ericsson.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-07Incremented CDT feature version to 8.6.0.Sergey Prigogin7-7/+7
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 Laskavaia5-25/+100
- 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 Ridge2-1/+10
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 Laskavaia3-21/+23
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-12-03Avoid ClassCastException in Codan Run jobAlena Laskavaia1-26/+27
Occasionally Run Code Analysis actions get text selection instead of structured selection so it is failing and logging CCE. Avoid this by non accepting other selections. Change-Id: Ia3583755547e5251a057290e506184656ebf2d5a Signed-off-by: Alena Laskavaia <elaskavaia.cdt@gmail.com> Reviewed-on: https://git.eclipse.org/r/37306 Reviewed-by: Sergey Prigogin <eclipse.sprigogin@gmail.com>
2014-09-13Bug 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-10Bug 353352 - CFG doesn't handle range-based for loopAlena Laskavaia2-11/+101
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-08-19Bug 437392 - Added the Launch Targets View.Doug Schaefer1-1/+4
This view lists all launch targets that could appear in the Target selector. It allows us to add context menus to them to perform operations, such as opening a terminal view to the target. It will also gives us a cleaner UI to add and remove targets Change-Id: I538a204ce6f25fd017573c50585ac45cdeeaf554 Reviewed-on: https://git.eclipse.org/r/31856 Tested-by: Hudson CI Reviewed-by: Jonathan Williams <jonwilliams@qnx.com> Reviewed-by: Doug Schaefer <dschaefer@qnx.com>
2014-08-14Proper reporting of unsupported statement types.Sergey Prigogin3-4/+52
Change-Id: I4e4745bf7661329873107e8849f6200577756891 Reviewed-on: https://git.eclipse.org/r/31725 Tested-by: Hudson CI Reviewed-by: Sergey Prigogin <eclipse.sprigogin@gmail.com> Tested-by: Sergey Prigogin <eclipse.sprigogin@gmail.com>
2014-08-13Cosmetics.Sergey Prigogin1-3/+3
2014-07-22Inremented branding plugin version to 8.5Sergey Prigogin7-7/+7
2014-06-21Bug 394521 Do Constant folding propagation when building CFGAlena Laskavaia3-5/+94
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 Marr2-15/+39
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 Laskavaia4-18/+78
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-23Added a TODO.Sergey Prigogin1-4/+3
2014-04-23Fixed a typo.Sergey Prigogin1-5/+5
2014-04-11Fixed compiler warnings.Sergey Prigogin1-1/+0
2014-04-09Missing NON-NLS stringsMarc Khouzam1-1/+1
Change-Id: Ia177b5ded46b3d6292af3e41951165c9d26b57cc Signed-off-by: Marc Khouzam <marc.khouzam@ericsson.com>
2014-04-09Remove unused API filtersMarc Khouzam1-11/+0
Change-Id: Ia3a53fec79973ccdec7d74d68fcc4f4621fe1cc7 Signed-off-by: Marc Khouzam <marc.khouzam@ericsson.com>
2014-04-07Bug 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-31Bug 431544 - UnsupportedOperationException inSergey Prigogin1-1/+2
ControlFlowGraphBuilder.build Change-Id: I91fd0b4e1a816b80a4bbad5e7a8ee1f61f9862a9 Reviewed-on: https://git.eclipse.org/r/24214 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-31Cosmetics.Sergey Prigogin2-97/+34
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-02-23Bug 428725 - Severity selection from Coda Analyses preferences shouldTeodor Madan1-1/+1
not allow not supported values. Create ComboBoxCellEditor with SWT.READ_ONLY style Change-Id: I5d1bbc94ffb6df17be092108c1e45133849cea3e Signed-off-by: Teodor Madan <teodor.madan@freescale.com> Reviewed-on: https://git.eclipse.org/r/22363 Tested-by: Hudson CI
2014-02-12Bug 428041 - NPE in ReturnCheckerSergey Prigogin1-78/+33
2014-01-24Enabled JDK 1.7.Sergey Prigogin28-46/+46
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 Prigogin7-7/+7
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-17Bug 424160: null checks for CodanPreferencesLoader.getProjectNodeAndrew Eidsness2-10/+12
CodanPreferencesLoader.getProjectNode will sometimes return null. This patch adds null checks before using the result. Change-Id: If4dd70e6673d5da62cc0b8e899fb5066bdaa4bef Signed-off-by: Andrew Eidsness <eclipse@jfront.com> Reviewed-on: https://git.eclipse.org/r/19873 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-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-12-06Cosmetics.Sergey Prigogin2-34/+9
2013-12-06Bug 400073 - Indexer runs out of memory.Sergey Prigogin1-8/+8
An attempt to reduce memory consumption.
2013-12-06Cosmetics.Sergey Prigogin1-8/+0
2013-12-01Bug 421900 - NPE in StatementHasNoEffectChecker.usesOverloadOperator()Nathan Ridge1-6/+13
Change-Id: I9099f3600cfdac492784087ef5993b353111eab8 Signed-off-by: Nathan Ridge <zeratul976@hotmail.com> Reviewed-on: https://git.eclipse.org/r/18468 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-10-09Fixed an NPE during shutdown.Sergey Prigogin1-0/+2
2013-09-06Bug 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-09-06Bug 380751 - Members of decltype types cannot be usedNathan Ridge1-3/+4
Change-Id: Ie6113d44647a222e063ab2ab27f3dc9d994f98fc Signed-off-by: Nathan Ridge <zeratul976@hotmail.com> Reviewed-on: https://git.eclipse.org/r/15451 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-28Bug 390782 - NPE in AssignmentToItselfChecker$1.isAssignmentToItselfSergey Prigogin1-9/+13

Back to the top