mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-04-29 19:45:01 +02:00
Code Issues Wiki Activity

[295895] DSF Disassembly view attempts to insert source even when 'Show Source' option is off

Browse source
This commit is contained in:
Anton Leherbauer 2009-11-26 08:27:39 +00:00
parent c753fd5b74
commit 214ac8648b
2 changed files with 72 additions and 57 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/debug/internal/ui/disassembly/DisassemblyPart.java
View file

@ -1752,7 +1752,7 @@ public abstract class DisassemblyPart extends WorkbenchPart implements IDisassem
return;
}
if (DEBUG) System.out.println("retrieveDisassembly "+getAddressText(startAddress)+" "+lines+" lines"); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
retrieveDisassembly(startAddress, endAddress, lines, true, false);
retrieveDisassembly(startAddress, endAddress, lines, fShowSource, false);
}
private void retrieveDisassembly(final BigInteger startAddress, BigInteger endAddress, final int linesHint, boolean mixed, boolean ignoreFile) {
@ -3346,6 +3346,14 @@ public abstract class DisassemblyPart extends WorkbenchPart implements IDisassem
} else {
fPCAnnotationUpdatePending = true;
updateInvalidSource();
if (fShowSource) {
Runnable doit = new Runnable() {
public void run() {
fDocument.invalidateAddressRange(fStartAddress, fEndAddress, true);
fGotoFramePending = true;
}};
doScrollLocked(doit);
}
}
}

119
dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/debug/internal/ui/disassembly/model/DisassemblyDocument.java
View file

@ -25,6 +25,8 @@ import org.eclipse.debug.core.sourcelookup.containers.LocalFileStorage;
import org.eclipse.jface.text.BadLocationException;
import org.eclipse.jface.text.BadPositionCategoryException;
import org.eclipse.jface.text.DefaultLineTracker;
import org.eclipse.jface.text.DocumentRewriteSession;
import org.eclipse.jface.text.DocumentRewriteSessionType;
import org.eclipse.jface.text.IDocument;
import org.eclipse.jface.text.IRegion;
import org.eclipse.jface.text.Position;
@ -1228,71 +1230,76 @@ public class DisassemblyDocument extends REDDocument {
}
public void deleteDisassemblyRange(BigInteger startAddress, BigInteger endAddress, boolean invalidate, boolean collapse) {
String replacement = invalidate ? "...\n" : null; //$NON-NLS-1$
int replaceLen = replacement != null ? replacement.length() : 0;
AddressRangePosition lastPos = null;
ArrayList<AddressRangePosition> toRemove = new ArrayList<AddressRangePosition>();
Iterator<AddressRangePosition> it = getModelPositionIterator(startAddress);
while (it.hasNext()) {
AddressRangePosition pos = it.next();
BigInteger posEndAddress = pos.fAddressOffset.add(pos.fAddressLength);
if (pos instanceof LabelPosition) {
if (!invalidate && pos.length > 0 && posEndAddress.compareTo(endAddress) > 0) {
DocumentRewriteSession session = startRewriteSession(DocumentRewriteSessionType.STRICTLY_SEQUENTIAL);
try {
String replacement = invalidate ? "...\n" : null; //$NON-NLS-1$
int replaceLen = replacement != null ? replacement.length() : 0;
AddressRangePosition lastPos = null;
ArrayList<AddressRangePosition> toRemove = new ArrayList<AddressRangePosition>();
Iterator<AddressRangePosition> it = getModelPositionIterator(startAddress);
while (it.hasNext()) {
AddressRangePosition pos = it.next();
BigInteger posEndAddress = pos.fAddressOffset.add(pos.fAddressLength);
if (pos instanceof LabelPosition) {
if (!invalidate && pos.length > 0 && posEndAddress.compareTo(endAddress) > 0) {
try {
int oldLength = pos.length;
pos.length = 0;
replace(pos, oldLength, null);
} catch (BadLocationException e) {
internalError(e);
}
}
pos = null;
} else if (pos instanceof SourcePosition) {
pos = null;
} else if (pos instanceof ErrorPosition) {
pos = null;
} else if (pos instanceof DisassemblyPosition) {
// optimization: join adjacent positions
if (collapse && lastPos != null
&& (invalidate || lastPos.fValid == pos.fValid)
&& lastPos.offset+lastPos.length == pos.offset) {
assert lastPos.fAddressOffset.add(lastPos.fAddressLength).compareTo(pos.fAddressOffset) == 0;
lastPos.length += pos.length;
lastPos.fAddressLength = lastPos.fAddressLength.add(pos.fAddressLength);
toRemove.add(pos);
if (!pos.fValid) {
fInvalidAddressRanges.remove(pos);
}
pos = null;
if (posEndAddress.compareTo(endAddress) < 0) {
continue;
}
}
}
if (lastPos != null) {
try {
int oldLength = pos.length;
pos.length = 0;
replace(pos, oldLength, null);
if (lastPos.length > 0 || replaceLen > 0) {
int oldLength = lastPos.length;
lastPos.length = replaceLen;
replace(lastPos, oldLength, replacement);
}
} catch (BadLocationException e) {
internalError(e);
}
}
pos = null;
} else if (pos instanceof SourcePosition) {
pos = null;
} else if (pos instanceof ErrorPosition) {
pos = null;
} else if (pos instanceof DisassemblyPosition) {
// optimization: join adjacent positions
if (collapse && lastPos != null
&& (invalidate || lastPos.fValid == pos.fValid)
&& lastPos.offset+lastPos.length == pos.offset) {
assert lastPos.fAddressOffset.add(lastPos.fAddressLength).compareTo(pos.fAddressOffset) == 0;
lastPos.length += pos.length;
lastPos.fAddressLength = lastPos.fAddressLength.add(pos.fAddressLength);
toRemove.add(pos);
if (!pos.fValid) {
fInvalidAddressRanges.remove(pos);
}
pos = null;
if (posEndAddress.compareTo(endAddress) < 0) {
continue;
if (pos == null && posEndAddress.compareTo(endAddress) >= 0) {
break;
}
lastPos = null;
if (pos != null) {
if (pos.fValid && invalidate) {
pos.fValid = false;
fInvalidAddressRanges.add(pos);
}
lastPos = pos;
}
}
if (lastPos != null) {
try {
if (lastPos.length > 0 || replaceLen > 0) {
int oldLength = lastPos.length;
lastPos.length = replaceLen;
replace(lastPos, oldLength, replacement);
}
} catch (BadLocationException e) {
internalError(e);
}
}
if (pos == null && posEndAddress.compareTo(endAddress) >= 0) {
break;
}
lastPos = null;
if (pos != null) {
if (pos.fValid && invalidate) {
pos.fValid = false;
fInvalidAddressRanges.add(pos);
}
lastPos = pos;
}
removePositions(CATEGORY_DISASSEMBLY, toRemove);
} finally {
stopRewriteSession(session);
}
removePositions(CATEGORY_DISASSEMBLY, toRemove);
if (DEBUG) checkConsistency();
}