Bug 360891 - [Architecture] Photran core, core.vpg, and UI should not depend on CDT
Summary: [Architecture] Photran core, core.vpg, and UI should not depend on CDT
Status: ASSIGNED
Alias: None
Product: PTP
Classification: Tools
Component: Photran.For Internal Use (show other bugs)
Version: 7.0   Edit
Hardware: All All
: P5 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Jeffrey Overbey CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-13 17:25 EDT by Jeffrey Overbey CLA
Modified: 2012-03-01 10:58 EST (History)
0 users

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 Jeffrey Overbey CLA 2011-10-13 17:25:47 EDT 
The only Photran plug-ins with dependencies on CDT should be
- cdtinterface and cdtinterface.vpg
- managed build plug-ins
- preprocessor.c
- automated tests which rely on these plug-ins

In particular,
- error parsers should be moved from photran.core into cdtinterface
- photran.core.vpg's spurious dependency on cdt.core should be removed
- the CDT references in photran.ui should be removed

Eventually, it would be ideal to eliminate the CDTBasedTextEditor class altogether, although that does not appear to be feasible in the short term.
Comment 1 Jeffrey Overbey CLA 2011-10-13 17:33:35 EDT 
The error parsers have been moved and the core.vpg dependency have been removed in HEAD.
Comment 2 Jeffrey Overbey CLA 2011-10-13 19:45:41 EDT 
The dependencies from photran.ui have also been removed in HEAD.

I was not able to remove the CDT dependencies from ui.vpg since it depends on FortranElement (for completion proposal images) and CCompositeReconcilingStrategy.

This should be targeted for 8.0, then closed.