Bug 186256

Summary: Need to implement View Model for Variables view.
Product: [Tools] CDT Reporter: Fran Litterio <francis.litterio>
Component: cdt-debug-dsf-gdbAssignee: cdt-debug-inbox <cdt-debug-inbox>
Status: RESOLVED DUPLICATE QA Contact: Marc Khouzam <marc.khouzam>
Severity: normal    
Priority: P3 Keywords: contributed
Version: 6.0   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Patch providing initial implementation of DSF view model components for Variables view (+committed, +logged)
cdtdoug: iplog+
Added type names support. (+committed, +logged) pawel.1.piech: iplog+

Description Fran Litterio CLA 2007-05-09 17:07:43 EDT
Build ID: I20070323-1616

Steps To Reproduce:
Stop in any function: no variables are displayed.

More information:
Comment 1 Fran Litterio CLA 2007-05-09 17:13:27 EDT
Created attachment 66568 [details]
Patch providing initial implementation of DSF view model components for Variables view (+committed, +logged)

This patch provides an initial implementation of classes VariableLocalsLayoutNode and VariableSubExpressionsLayoutNode to make the Variables view functional.
Comment 2 Randy Rohrbach CLA 2007-05-09 17:25:40 EDT
Applied Frans patch. Randy Rohrbach
Comment 3 Randy Rohrbach CLA 2007-05-09 17:33:51 EDT
Tested and see it is working and therefore completed.
Comment 4 Fran Litterio CLA 2007-05-11 12:21:38 EDT
Reopening this bug.  There is more work to be done.
Comment 5 Fran Litterio CLA 2007-05-11 13:33:44 EDT
Created attachment 66914 [details]
Added type names support. (+committed, +logged)

This patch adds type names support to the Variables view.  It also removes the unneeded method getNaturalValue() from interface IExpressions.IExpressionDMData.  It also condenses several "$NON-NLS-X$" comments into a single @SuppressWarnings annotation.
Comment 6 Pawel Piech CLA 2007-05-25 16:09:01 EDT
Changed component to MI, to help with bug searches.
Comment 7 Pawel Piech CLA 2007-05-25 19:06:33 EDT
Comment on attachment 66914 [details]
Added type names support. (+committed, +logged)

I applied this patch a week before coding camp.  But I didn't commit the suppress warnings changes.  We want to use the //NON-NLS comments for use with wizards.
Comment 8 Pawel Piech CLA 2007-06-19 14:36:08 EDT
This bug was used for the original variables view model contribution.  For the rest of the work, let's use the original enhancement bug 159695.

*** This bug has been marked as a duplicate of bug 159695 ***