Bug 178831 - API inconsistency: AbstractCDebuggerTab must be API
Summary: API inconsistency: AbstractCDebuggerTab must be API
Status: RESOLVED WONTFIX
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug-cdi (show other bugs)
Version: 4.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: cdt-launch-inbox@eclipse.org CLA
QA Contact:
URL:
Whiteboard:
Keywords: api
Depends on:
Blocks: 257402
  Show dependency tree
 
Reported: 2007-03-22 12:57 EDT by Martin Oberhuber CLA
Modified: 2017-10-23 13:38 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Oberhuber CLA 2007-03-22 12:57:25 EDT 
The RSE remotecdt plugin extends the CDebuggerTab, which is API.
But CDebuggerTab derives from AbstractCDebuggerTab, which is not API (internal).

This is inconsistent. If a class is API, then all the parent classes it extends must also be API. Therefore, AbstractCDebuggerTab should be moved to
the org.eclipse.cdt.launch.ui package.

To be concrete, our extension wants to call protected method loadDebuggerCombo() which is declared in AbstractCDebuggerTab.
Comment 1 Martin Oberhuber CLA 2008-12-03 13:13:56 EST 
Galileo has API cleanliness as a must-have requirement, so I'd like to re-initiate discussion about this. See also bug 257402.
Comment 2 Jonah Graham CLA 2017-10-23 13:38:49 EDT 
(this is part of a batch change)

The CDI debug implementation has been removed in CDT 9.0. Please see bug 484900 and the entry on the New and Noteworthy page https://wiki.eclipse.org/CDT/User/NewIn90#API_modifications