Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 323221 - [Memory Browser] Error indicator of address input field does not clear when debug context changes
Summary: [Memory Browser] Error indicator of address input field does not clear when d...
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-memory (show other bugs)
Version: 7.0   Edit
Hardware: PC Windows XP
: P3 minor (vote)
Target Milestone: ---   Edit
Assignee: Randy Rohrbach CLA
QA Contact: Ted Williams CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-20 05:42 EDT by Anton Leherbauer CLA
Modified: 2011-05-12 17:05 EDT (History)
2 users (show)

See Also:
Randy.Rohrbach: review? (pawel.1.piech)


Attachments
Branch 6_0 patch (1.05 KB, text/plain)
2010-09-14 07:57 EDT, Andre St. Laurent CLA
no flags Details
Branch 7_0 patch (1.04 KB, patch)
2010-09-14 07:57 EDT, Andre St. Laurent CLA
cdtdoug: iplog+
Details | Diff
HEAD patch (1.14 KB, patch)
2010-09-14 08:14 EDT, Andre St. Laurent CLA
cdtdoug: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Anton Leherbauer CLA 2010-08-20 05:42:30 EDT
When an expression entered in the address field is invalid in the current debug context an error indicator appears to the left of the input field. 
Selecting a debug context where the expression is valid the memory browser updates the content, but the error indicator remains.
Comment 1 Andre St. Laurent CLA 2010-09-14 07:57:01 EDT
Created attachment 178809 [details]
Branch 6_0 patch

This patch may be applied to branch 6_0[_2] of org.eclipse.cdt.debug.ui.memory.memorybrowser.MemoryBrowser.java
Comment 2 Andre St. Laurent CLA 2010-09-14 07:57:53 EDT
Created attachment 178810 [details]
Branch 7_0 patch

This patch may be applied to branch 7_0 of
org.eclipse.cdt.debug.ui.memory.memorybrowser.MemoryBrowser.java
Comment 3 Andre St. Laurent CLA 2010-09-14 08:14:26 EDT
Created attachment 178812 [details]
HEAD patch

This patch may be applied to HEAD, to the following file: 
org.eclipse.cdt.debug.ui.memory.memorybrowser.MemoryBrowser.java
Comment 4 Randy Rohrbach CLA 2010-09-20 15:10:37 EDT
Folks

   I am taking this one to apply Andre's fix.

Randy
Comment 5 Randy Rohrbach CLA 2010-09-20 15:17:05 EDT
Pawel

   I applied these patches to HEAD and the 7.0 CDT branch, please review

Randy