Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 325150

Summary: edc - ARM - disassembly - fix for "duplicate line" and "missing line" bugs
Product: [Tools] CDT Reporter: Kirk Beitz <kirk.beitz>
Component: cdt-debug-edcAssignee: Ken Ryall <ken.ryall>
Status: RESOLVED FIXED QA Contact: Ken Ryall <ken.ryall>
Severity: normal    
Priority: P3    
Version: 7.0.1   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:

Description Kirk Beitz CLA 2010-09-13 14:12:00 EDT 
ARMDisassembly.java was duplicating effort by figuring out the ARM-or-Thumb mode in both the getMixedInstructions() and getInstructions() functions … and in certain circumstances, this was resulting in either duplicate assembly entries or else missing lines in the code.

the code in ARMDisassembly.java has been re-factored so that if getMixedInstructions() performs the getModeAndRange() prior to calling getInstructions(), it passes the range, and getInstructions() now uses the pre-calculated range if non-null.  (if null, it calculates the range as it had before, so that external callers get the same behavior.)

(this bugzilla entry is a placeholder for changes to be merged that have
already been committed and tested in the nokia database.  presumably such a
merge will also go on the CDT_7_0 branch for 7.0.2 after cdt 7.0.1 is
released.)
Comment 1 Kirk Beitz CLA 2010-10-06 22:45:32 EDT 
resolved with 2010.oct.04 edc merge commits by ken.ryall@nokia.com