Skip to main content
AgeCommit message (Collapse)AuthorFilesLines
2021-09-26[releng] Prepare for CDT 10.5.0Jonah Graham1-1/+1
Change-Id: I9b72edf4a0ff0771a147934dd7f1c92ba1ee799a
2021-07-24Bug 574247: Test for order of elements in outline viewJonah Graham1-0/+21
Change-Id: I75593befecdc19ddee137830a457c8dbfb486a03
2021-06-12[releng] Bump to CDT version 10.4.0Jonah Graham1-1/+1
Change-Id: I0807f46d9ebb8bb579b687366cdb19e40935a9ec
2021-04-04Bug 572552 - CDT releng: update versions to 10.3.0Alexander Fedorov1-1/+11
Added missed license headers Updated baseline for the parent pom to CDT 10.2 Updated version for pom.xml to 10.3 Updated version for features and bundles to CDT 10.3 Updated copyright for to be 2021 Incremented version + 100 where needed Change-Id: I79666fcc0402fee6607499d7dce1eaf87a5f446d Signed-off-by: Alexander Fedorov <>
2021-02-01Bug 569839: Provide a new JUnit5 base and utility classesJonah Graham1-0/+1
Change-Id: I8682f4702cfa0cad7d0452ca48d1ab74eeb1dbdb
2020-12-21Bug 569839: Ensure that behaviour is same running tests without suitesJonah Graham54-229/+284
This change renames all suites to be called ...TestSuite so that the default includes don't pick them up. With the name ...Tests it means that those tests referenced by the suite are run multiple times. Tests which relied on being run as part of a TestSuite have been refactored to run as individual tests by moving the logic from the suite into the test (e.g. CommentHandlingTestSuite) Tests that were not runnable have been renamed from ...Test to ...Tester to match a convention already used in CDT. Lots of tests did not use standard name (i.e. didn't end in no Test) so this was fixed. Many tests were really abstract tests, so the missing abstract was added, e.g. ArrayTests The default excludes in Maven behaviour differently than may be expected, so rather than remove static inner test classes, update the excludes, see pom.xml change Change-Id: Ia91e12fe76c3ec2d914463a28400d21b9daf1910
2020-12-21Bug 569839: Add DefaultCCommentAutoEditStrategyTest to testsuiteJonah Graham3-10/+16
This test started failing (and wasn't in testsuite previously) because it had significant trailing whitespace in some tests. The fix is to use ${whitespace_eol} which I have also added to some of the javadocs to make it easier to find next time. Change-Id: Ib364d8a400bcdeb21445dde8428e0fd4c77db990
2020-12-21Bug 569839: Restore the cleanupProjectJonah Graham5-5/+5
This style is the JUnit3 way of doing JUnit4 @AfterClass. It was commented out as part of Bug 564002 (commit 9e303185f96). Change-Id: I56e2754f3cce447c47546c91cdc16b302e18b4a6
2020-12-19[releng] Increment feature and related versions to 10.2.0Jonah Graham1-1/+1
Change-Id: Icd135a588920fa25fd846dc285db5f83e5ac7037
2020-11-18Bug 568728: Drop execute permission on filesTorbjörn Svensson2-0/+0
Add enforecment script that verifies that only specified file types are allowed to be mared as executable. Change-Id: I4b40e3a46b03bfc78a9dcd52e8bf29cb079b7f3b Signed-off-by: Torbjörn Svensson <>
2020-10-12Bug 567778 - [Include Browser] Cannot open the Include Browser on file ↵Marc-Andre Laperle2-4/+44
outside source folder CoreModelUtil.findTranslationUnit only returns CElement in the populated CModel of a project. This shouldn't change as a large majority of client code need to see the model this way and not consider files that are outside source folders. So for a file not under a source folder (and therefore not in the CModel), we can just create a new translation unit instance for it. This is actually how the editor deals with it too. Change-Id: I8898822e94cac8562edcc0a726fdd8680119faca Signed-off-by: Marc-Andre Laperle <>
2020-10-06[releng] Bump version to 10.1.0Jonah Graham1-1/+1
Change-Id: I39f7c379c0b43c086a25b047b3af564f6f24d670
2020-10-05Bug 567261 - Revert "541239: [C++17] Implement parser support for deduction ↵Marc-Andre Laperle1-37/+0
guides" This reverts commit bcb955a32104da02f3748dd3ef080cbcb43c646b. Change-Id: Iaf6ba398e439e3d1da1e3a3b0247c7b24bbfb057
2020-09-27Bug 566918 - [C++17] Support for __has_include (standard) andMarc-Andre Laperle2-3/+4
__has_include_next (extension) __has_include evaluates whether of the header name passed as parameter exists. This can only be evaluated as part of a #if directive. Interestingly, it also has to be reported as defined, i.e. #if defined(__has_include) or #ifdef. In order to report this as defined, this implementation adds it as a macro but during macro expansion, it's actually converted as a dedicated token type. Then this token gets evaluated during normal preprocessor expression evaluation. In order to parse header names, there were several options. The main problem is that header tokens (tQUOTE_HEADER_NAME, tSYSTEM_HEADER_NAME) are actually produced by the Lexer as part of a special mode (setInsideIncludeDirective) set during the handling of #include. For expression evaluation, the tokens are already generated without setInsideIncludeDirective therefore we only have plain string and < > tokens. One approach would be to generate header tokens "earlier" than executing we need to track a new state while fetching token to configure the Lexer (setInsideIncludeDirective) when in the context of an __has_include. There are also complications due to macro expansion within the __has_include where after one expansion, we don't have a lexer in the context anymore, introducing more changes. Another approach would be to remove the Header token creation from the Lexer itself and let the preprocessor assemble the tokens into an header string, in both cases of #include and __has_include. This mostly works and is the approach used in Clang, but the problem is that whereas Clang keeps track of leading spaces of tokens, CDT doesn't. This means with such change that CDT would now allow #include < iostream > (notice the white space). I think this is too big of a downside and also too big of a change to introduce this handling of whitespace at the token level. The approach used here is more conservative and isolated but also shares less common logic with #include processing. The non-header token (string, <, etc) are assembled into a header string only in the case of a __has_include. So a downside will be that #include and __has_include will be inconsistent in regards of leading/trailing space parsing but I feel like this is better than making #include more permissive. Change-Id: I5b9f5c616c8d999e0c916a85b41f96e20037b651 Signed-off-by: Marc-Andre Laperle <>
2020-08-31Bug 558809: Handle cases where Oomph corrupts \0 char in preferenceJonah Graham2-0/+83
Some CDT preferences use \0 as a separator in preferences. Somewhere in the Oomph preference synchronizer stack there is, or was, a place that failed to escape/unescape preferences with encoded \0 properly. CDT would then fail to parse the preference and an exception would be raised, causing code completions and the editor to be broken. This patch hardens the CDT code to: (1) Allow an escaped \0 to be used as a separator on read (Oomph uses ${0x0}) (2) Handle NumberFormatExceptions gracefully. In this case that means showing user a pop-up that their completion preferences are empty and offering to reset them, or edit them in preference page. This UI logic already existed, so all the new code has to do on failed parse is return a list of all disabled completions. Change-Id: Ibf3b05c0855bb96c195ca43139a50c27a2a90c7e
2020-08-30Bug 566511: [releng] Add missing UTF-8 encoding for all CDT projectsJonah Graham1-0/+2
Includes updates to code cleanliness script Change-Id: Ic495124285e3e002993f34dc12be5ad7337fe13b
2020-08-28Fixed "The value for X attribute is not externalized" warningTorbjörn Svensson2-2/+6
Change-Id: Ib2f0d85a0428a1f59cc7ccab6ebb8fd91ffa2a41 Signed-off-by: Torbjörn Svensson <>
2020-08-24Bug 562494: Update BREE to Java 11Jonah Graham3-12/+42
Change-Id: I0c91aafc0cb1b179936acbd4c9df2d961899e3fa
2020-08-13Bug 564002: Restore the default pattern of the surefire-pluginMartin Weber24-425/+251
Sets the pattern to the default of upcoming tycho 2.0 in advance. Signed-off-by: Martin Weber <> Change-Id: I31b3fc733d0cb888fbf6f566995ce2043f6cd621
2020-06-15Bug 564273 - Fix format lambda expressions without parenthesisMarco Stornelli1-0/+15
Change-Id: I918ca05d75ca4e8cba7501e232d4e6b05e434f06
2020-06-02Bug 563864: By default fail build on test failures when building CDT with mavenJonah Graham1-1/+0
Change-Id: I9bd3db3850cc45abb0ea53f8885a331c7f327887
2020-06-01Bug 487990 - Fix format of variadic functionsMarco Stornelli1-0/+11
Change-Id: I802d032f733247178db46c8fe43fdb9350555509
2020-05-09Bug 562997 - Switch CDT doxygen to use new Equinox preferences APIAlexander Fedorov1-2/+2
Switched from "org.eclipse.cdt.core.options" to "org.eclipse.core.runtime.preferences" Change-Id: Ie01d955079a7a54875270bb2d259c9232d30e7ea Signed-off-by: Alexander Fedorov <>
2020-05-09Bug 562896 - Fix qualified name in return type for implement methodMarco Stornelli1-0/+17
Change-Id: Ie1ff2e401091655c6ecc2a7393dac19cea872a0e
2020-05-04Bug 562722 - Fix noexcept using implement methodMarco Stornelli2-1/+18
Change-Id: I3ac99f916883fa4be55af29e51b63bc8aa40a63f
2020-05-04Bug 562723 - Added support for noexcept in the formatterMarco Stornelli1-0/+24
Change-Id: I021934657842868c196320f4e126217ab799c07c
2020-04-22Bug 562292 - Fix method definition position for override methodMarco Stornelli1-0/+62
The position of namespaces didn't take into account because the declaration doesn't exist yet when we use this kind of refactoring, so the find method of MethodDefinitionInsertLocationFinder didn't look for namespaces. Change-Id: I839194879c41f86653c837ca83a306ea1840c1d0
2020-04-21Bug 562368 - Fix doxygen template documentationMarco Stornelli1-0/+15
Change-Id: If0763d66fab13bc1cc7445363368df4c53546c24
2020-04-16Bug 562181: Fix auto indent after constructor with access specifiersAndrey Mozzhuhin1-0/+19
Some heuristics in skipToStatementStart require knowledge of previous token. It is initialized with value from a fToken, but it current value may be invalid because looksLikeMethodDecl changes fPossition value without adjusting a fToken. Using invalid value may lead to false triggering of heuristics. Restore fToken value when change fPossition in looksLikeMethodDecl to prevent this. Change-Id: If0c2c0577c89a983e1479587409f293c3d0db7be Signed-off-by: Andrey Mozzhuhin <>
2020-04-15Bug 562125: Fix auto indent of function call with scope qualifiersAndrey Mozzhuhin1-0/+15
A function call may be mistakenly interpreted in looksLikeMethodDecl() as a method declaration. This was due to simplified processing of functions with a scope qualifiers in the name. Now methods with a scope qualifier are handled similarly to methods without them. Change-Id: Id3075d3387fdf9c4ae2d0dffa6cdf923fd1ef9d5 Signed-off-by: Andrey Mozzhuhin <>
2020-04-13Bug 516393: Fix scan of double char tokens in CHeuristicScannerAndrey Mozzhuhin2-0/+29
Scanning of double char tokens (::, >>, >=, <<, <=, ->) is broken in nextToken(). In each case, peekNextChar() was used to get second character, but scanner position was already on second char and peekPreviousChar() need to be used. Change-Id: Ibd447c7cde8783e8ffe547d5f9bc09d11c1c60a7 Signed-off-by: Andrey Mozzhuhin <>
2020-04-12Bug 561993 - Remove dependency to from CDT UIAlexander Fedorov3-6/+6
Use JVM classes Rework WorkingSetProxy#getSearchKey Change-Id: Icda4b5b791259150dbf436546c2096ea728e0da0 Signed-off-by: Alexander Fedorov <>
2020-04-11[releng] Bump version to 10.0.0Jonah Graham1-1/+1
Change-Id: I32f6f61835bdbad3cffd713965045c5097c8619f
2020-03-30Bug 561559 - Fix formatting def capture lambda expressionsMarco Stornelli1-0/+19
We missed a space before the closing brackets if the proper option was selected. Change-Id: Ibbb09c3c961dc1b5e22aaa65ffb5d9878c2bb08b
2020-03-22[releng] Bump CDT version to 9.12.0Marco Stornelli1-1/+1
org.eclipse.cdt.debug.gdbjtag is bumped because of Bug 561343 Change-Id: I6dac283b7e9093662f57ac5c804021c4201ad6f1
2020-03-13541239: [C++17] Implement parser support for deduction guidesFelix Morgner1-0/+37
Change-Id: I1683583af981d276b346ba8470a9044cfd8921cd Task-Url: Signed-off-by: Felix Morgner <> Also-by: Marco Stornelli <>
2020-03-02Revert "Bug 560614 - Drop dependency on ICU4J"Jonah Graham3-2/+4
This reverts commit 87b627bf02eaab28cdb64f6ebe203ac64d18fe94. Change-Id: I54ea1ac766437deb174eeba571a7a5094c180828
2020-03-02Bug 560614 - Drop dependency on ICU4JAlexander Kurtakov3-4/+2
Straightforward conversion. Still usages left that need deeper investigation how to be done proper. Some long time commented code removed as java formatter breaks trailing whitespaces on save. Change-Id: If74259bed5735b0d4cc98fc2cfa609c9c53c80c9 Signed-off-by: Alexander Kurtakov <>
2020-02-01Bug 559669 - Fix formatter on/off tagsMarco Stornelli1-0/+17
Change-Id: Ia481703a0360ec5741d2fb079a75aa13a33bff39 Signed-off-by: Marco Stornelli <>
2020-01-30Bug 559545 - Fix formatter with enums with standard attributesMarco Stornelli1-0/+9
Change-Id: I993e773792105e22b9c2f703488e7ff211ed4068 Signed-off-by: Marco Stornelli <>
2020-01-30Bug 434677 - Fix implement method insert position with namespacesMarco Stornelli1-0/+22
If the correct namespace existed before the refactoring in the translation unit, refactoring process just ignored it. Change-Id: I9d6bd301807bb2d3f83f74ef772395d3470cf8bd Signed-off-by: Marco Stornelli <>
2020-01-25Bug 452809 - Implement method fully qualified decl specifierMarco Stornelli1-0/+20
If an enum, a struct or a class is defined inside a class, the implementation must qualify the declaration specifier. Change-Id: I1e9450c7b165a25c4a745c50fe2e0c06e28775d0 Signed-off-by: Marco Stornelli <>
2020-01-22Bug 559193 - Rework DoxygenPreferences to encapsulate constantsAlexander Fedorov2-23/+23
Part 2 (please refer to the Bug 559193 regarding the reasons to rework): - Doxygen option read-only API for clients - Doxygen option storage API for configuration UI - OSGi component for Doxygen API implementation Change-Id: Ia97ade397f1040087898aeb3b50872473d3c0631 Signed-off-by: Alexander Fedorov <>
2020-01-20Bug 384793 - Fix implement function refactoring with var argsMarco Stornelli1-0/+13
Change-Id: I420a6a8197ff343a5719d8cc3d85eeb9ea889dfa Signed-off-by: Marco Stornelli <>
2020-01-19Bug 510289 - Fix implementation refactoring with template parametersMarco Stornelli1-0/+39
The implementation of methods with parameter declaration or with template template parameters didn't work. Change-Id: I783dedc5ffecd6721293d52f13548fd9e73999bc Signed-off-by: Marco Stornelli <>
2020-01-16Bug 552334 - Generate enum doxygen comments according to comment styleMarco Stornelli2-10/+85
Change-Id: I49f8188cc9c5421c90a48fb96a5b476de10ef251 Signed-off-by: Marco Stornelli <>
2020-01-15Bug 333134 - Add options to configure doxygen behaviorAlexander Fedorov1-23/+22
Reworked UI code to its natural contract. Switched java 8 to compact the code Encapsulated scope inside the DoxygenPreferences Change-Id: I534b8d4f7c2d6e7674b132a10ef3514e61f2c1ed Signed-off-by: Alexander Fedorov <>
2020-01-14Bug 333134 - Add options to configure doxygen behaviorMarco Stornelli1-0/+85
Change-Id: Idf083d6e0fdf80a848412d610cb41da1d379aeaa Signed-off-by: Marco Stornelli <>
2020-01-09Bug 551817: Write Kichwa Coders consistently in Copyright lineJonah Graham4-4/+4
Choice is one of: Kichwa Coders Ltd Kichwa Coders Canada Inc. depending if contributed by UK or Canadian company Change-Id: I5c9fb96ea6abf97858a6896911a71fa68b8400b1
2020-01-08Bug 558923: Syntax coloring for binary integer literals from C++14Andrey Mozzhuhin2-2/+16
Change-Id: Ib3b260b6638e689884a28037d1a814b737846867 Signed-off-by: Andrey Mozzhuhin <>

Back to the top