Notice: this Wiki will be going read only early in 2024 and edits will no longer be possible. Please see: https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/wikis/Wiki-shutdown-plan for the plan.
Difference between revisions of "CDT/Obsolete/APIBreakages"
(→Proposed API-breaking changes) |
(→Proposed API-breaking changes) |
||
Line 32: | Line 32: | ||
** UPC, XLC plugins may need them, which are used by PTP | ** UPC, XLC plugins may need them, which are used by PTP | ||
** Can we move it to PTP? | ** Can we move it to PTP? | ||
+ | ***'''EclipeCon2016 update: Doug is following-up [https://dev.eclipse.org/mhonarc/lists/cdt-dev/msg30275.html here]''' | ||
* p2 plugin '''Doug will decide what to do here''' | * p2 plugin '''Doug will decide what to do here''' | ||
** Could it be part of p2 itself? | ** Could it be part of p2 itself? | ||
+ | ***'''EclipeCon2016 update: Doug will delete''' | ||
* o.e.cdt.core.native (Spawner) '''Doug''' | * o.e.cdt.core.native (Spawner) '''Doug''' | ||
** Used by o.e.remote but CDT uses o.e.remote => circular | ** Used by o.e.remote but CDT uses o.e.remote => circular | ||
** Spawner should be moved lower than CDT. Where? Maybe in CDT but be independent. That still solves 'circular' dependency. | ** Spawner should be moved lower than CDT. Where? Maybe in CDT but be independent. That still solves 'circular' dependency. | ||
** native should have its own pom/build | ** native should have its own pom/build | ||
+ | ***'''EclipeCon2016 update: Want to move to a IDE Commons project. Until then, we can just split the build. Not requiring API-breakage''' | ||
* Remove IA64, QNX as they are not available from platform '''Doug''' | * Remove IA64, QNX as they are not available from platform '''Doug''' | ||
+ | **'''EclipeCon2016 update: Can remove. Patch posted [here]''' | ||
+ | |||
+ | |||
* Remove o.e.linuxtools.cdt.autotools.core '''Jeff''' | * Remove o.e.linuxtools.cdt.autotools.core '''Jeff''' | ||
+ | **'''EclipeCon2016 update: Done''' | ||
* Merge cross-GCC with regular-GCC | * Merge cross-GCC with regular-GCC | ||
+ | **'''EclipeCon2016 update: Won't have time''' | ||
* Cleanup/Rename CDI* classes used by DSF '''Marc K''' | * Cleanup/Rename CDI* classes used by DSF '''Marc K''' | ||
+ | **'''EclipeCon2016 update: Keep old classes but make then stubs that point to a properly named set of classes. Non API-breaking''' | ||
* C/C++ Problem Markers - add API to deal with column information '''Alena''' | * C/C++ Problem Markers - add API to deal with column information '''Alena''' | ||
− | + | **'''EclipeCon2016 update: We believe this was done''' | |
* Rewrite error parsers to use a better regex library (java.regex.Pattern is part of API now) because build is horribly slow from IDE strictly because of error parsers. '''Doug, as part of the new build effort''' | * Rewrite error parsers to use a better regex library (java.regex.Pattern is part of API now) because build is horribly slow from IDE strictly because of error parsers. '''Doug, as part of the new build effort''' | ||
** Need to fix API | ** Need to fix API | ||
** Be careful with performance | ** Be careful with performance | ||
− | + | ***'''EclipeCon2016 update: Won't have time''' | |
* Rewrite ui for debug tabs, all current ui controls are APIs... | * Rewrite ui for debug tabs, all current ui controls are APIs... | ||
** Check with list if anyone is against such changes | ** Check with list if anyone is against such changes | ||
− | + | ***'''EclipeCon2016 update: Won't have time''' | |
* Java 8 changes '''Ok for CDT plugins to use Java 8. Migration of each plugin when/if needed''' | * Java 8 changes '''Ok for CDT plugins to use Java 8. Migration of each plugin when/if needed''' | ||
− | + | **'''EclipeCon2016 update: Would be nice to move all plugins. Let's write a script to do this whenever we need for different java versions. Non API-breaking.''' | |
* Make (some) dsf.ui and dsf.gdb.ui classes as API. | * Make (some) dsf.ui and dsf.gdb.ui classes as API. | ||
** Pay attention to good extensibility for some of these classes. Probably best to makes changes to address specific use-cases | ** Pay attention to good extensibility for some of these classes. Probably best to makes changes to address specific use-cases |
Revision as of 23:58, 7 March 2016
CDT 9.0
This page is attempt to collect possible API breaking proposals for next CDT release
Proposed API-breaking changes
- Remove all methods, interfaces and classes that have been deprecated since around 2 years CDT 8.?.0 and are not currently used in CDT code.
- Marc K to do the DSF/DSF-GDB part
- Improves readability
- Reduces maintenance
- Could be contributions from a hackathon
- Suggested rules (not mandatory): something deprecated for around 2 years and not used by CDT itself
- EclipeCon2016 update: Some of this cleanup has been done but not extensively.
- Remove support for old Apple GDB. Alvaro will have a look
- EclipeCon2016 update: Done
- Delete CDI plugins Doug will nuke it
- EclipeCon2016 update: Jonah has a patch but he mentioned having to fix some things before it is commitable.
- Clean up CApplicationLaunchShortcut to no longer chooseDebugConfig() which is a CDI concept Marc will look at that
- EclipeCon2016 update: Can be done once CDI is removed. Not requiring API-breakage
- Cleanup old-style projects Marc-Andre will head this
- EclipeCon2016 update: Marc-Andre has a patch he will try to complete
- Our tests still use old-style and migrating them is not trivial
- EclipeCon2016 update: These tests are essential and migrating them will take time. Will not happen.
- Remove code that migrates from .cdtproject (pre CDT 4.0)
- EclipeCon2016 update: Can be done. Maybe Marc-Andre?
- Removing UI that is not used
- EclipeCon2016 update: Can be done. Marc-Andre will have a look.
- Old scanner discovery (Language setting provider)
- At least we should not show both in the UI to a user
- EclipeCon2016 update: Doug will remove the element from the UI
- At least we should not show both in the UI to a user
- What about LR parser?
- Need to follow-up with IBM
- UPC, XLC plugins may need them, which are used by PTP
- Can we move it to PTP?
- EclipeCon2016 update: Doug is following-up here
- p2 plugin Doug will decide what to do here
- Could it be part of p2 itself?
- EclipeCon2016 update: Doug will delete
- Could it be part of p2 itself?
- o.e.cdt.core.native (Spawner) Doug
- Used by o.e.remote but CDT uses o.e.remote => circular
- Spawner should be moved lower than CDT. Where? Maybe in CDT but be independent. That still solves 'circular' dependency.
- native should have its own pom/build
- EclipeCon2016 update: Want to move to a IDE Commons project. Until then, we can just split the build. Not requiring API-breakage
- Remove IA64, QNX as they are not available from platform Doug
- EclipeCon2016 update: Can remove. Patch posted [here]
- Remove o.e.linuxtools.cdt.autotools.core Jeff
- EclipeCon2016 update: Done
- Merge cross-GCC with regular-GCC
- EclipeCon2016 update: Won't have time
- Cleanup/Rename CDI* classes used by DSF Marc K
- EclipeCon2016 update: Keep old classes but make then stubs that point to a properly named set of classes. Non API-breaking
- C/C++ Problem Markers - add API to deal with column information Alena
- EclipeCon2016 update: We believe this was done
- Rewrite error parsers to use a better regex library (java.regex.Pattern is part of API now) because build is horribly slow from IDE strictly because of error parsers. Doug, as part of the new build effort
- Need to fix API
- Be careful with performance
- EclipeCon2016 update: Won't have time
- Rewrite ui for debug tabs, all current ui controls are APIs...
- Check with list if anyone is against such changes
- EclipeCon2016 update: Won't have time
- Check with list if anyone is against such changes
- Java 8 changes Ok for CDT plugins to use Java 8. Migration of each plugin when/if needed
- EclipeCon2016 update: Would be nice to move all plugins. Let's write a script to do this whenever we need for different java versions. Non API-breaking.
- Make (some) dsf.ui and dsf.gdb.ui classes as API.
- Pay attention to good extensibility for some of these classes. Probably best to makes changes to address specific use-cases
- We will only do this upon request and if there is a good reason
- Separate the Run launch delegate into its own delegate. The Run delegate should use LocalRunLaunchDelegate.java instead of LocalCDILaunchDelegate.java Marc
- In the debugger, make thread a string instead of an integer. GDB may move to use strings. See IMIExecutionDMContext Marc
- MIBreakpoint MI uses an int as an id. It should be a string. Marc
- Changes for Grouping feature and its use of IContainer... Containers may no longer be directly mapped to processes. Marc
- Replace IRunControl.isStepping() and IRunConrol.isSuspended() with asynchronous versions???
- Revisit the CDT Breakpoint action framework Mikhail
Proposed process
- Mark classes that will disappear as Deprecated. Can we do it on the Package? Maybe using javadoc. Do this ASAP to warn people.
- Select an API-breaking change to work on
- Estimate the impact (code changes) of the change on the APIs
- Communicate that impact to the cdt-dev list to notify possible impacted parties