| Summary: | [disassembly] Multiple instruction pointers | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Tools] CDT | Reporter: | Patrick Chuong <pchuong> | ||||
| Component: | cdt-debug | Assignee: | Patrick Chuong <pchuong> | ||||
| Status: | RESOLVED FIXED | QA Contact: | Ken Ryall <ken.ryall> | ||||
| Severity: | normal | ||||||
| Priority: | P3 | CC: | aleherb+eclipse, cdtdoug, malaperle, pawel.1.piech | ||||
| Version: | 8.0 | Flags: | aleherb+eclipse:
review+
|
||||
| Target Milestone: | 8.1.0 | ||||||
| Hardware: | PC | ||||||
| OS: | Windows 7 | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
|
Description
Patrick Chuong
Created attachment 208946 [details]
patch
Toni, is this approach acceptable? In my debugger, I override DisassemblyBackendDsf to add additional PC annotation to the model.
(In reply to comment #1) > Toni, is this approach acceptable? In my debugger, I override > DisassemblyBackendDsf to add additional PC annotation to the model. Looks reasonable. Why do you define the class ExtendedPCAnnotationModel? It does not add any value. Please update the year in the copyright headers (2011 -> 2012). Thanks Toni, I have removed the ExtendedPCAnnotationModel class and updated the copyright. I also made an additional check for nullpointer for contextBackend within the DisassemblyPart class. |