X-Git-Url: https://mograsim.net/gitweb/?a=blobdiff_plain;f=plugins%2Fnet.mograsim.plugin.core%2Fsrc%2Fnet%2Fmograsim%2Fplugin%2Fviews%2FSimulationView.java;h=f81fb4b0423506905912edc018096c00c3f2739f;hb=13577856cd85c46f2cd4ad956332697bc820f425;hp=9f9f60b410dc51f75a5b54b0e12b6881934acbb0;hpb=240c1cb91e7ef820c25652c75187f0ece79723d6;p=Mograsim.git

diff --git a/plugins/net.mograsim.plugin.core/src/net/mograsim/plugin/views/SimulationView.java b/plugins/net.mograsim.plugin.core/src/net/mograsim/plugin/views/SimulationView.java
index 9f9f60b4..f81fb4b0 100644
--- a/plugins/net.mograsim.plugin.core/src/net/mograsim/plugin/views/SimulationView.java
+++ b/plugins/net.mograsim.plugin.core/src/net/mograsim/plugin/views/SimulationView.java
@@ -65,7 +65,7 @@ public class SimulationView extends ViewPart
 	{
 		controlsToDisableWhenNoMachinePresent = new HashSet<>();
 		memCellListener = a -> instPreview.refresh();
-		// TODO could this be a breakpoint?
+		// TODO use Step Over instead
 		clockObserver = o ->
 		{
 			if (((CoreClock) o).isOn())
@@ -148,7 +148,6 @@ public class SimulationView extends ViewPart
 		simSpeedScale.addListener(SWT.Selection, e ->
 		{
 			double speed = Math.pow(SIM_SPEED_SCALE_STEP_FACTOR, simSpeedScale.getSelection() - SIM_SPEED_SCALE_STEPS);
-			// TODO: disable when debugTarget is not set
 			debugTarget.setExecutionSpeed(speed);
 		});
 
@@ -175,7 +174,8 @@ public class SimulationView extends ViewPart
 
 	private void addInstructionPreviewControlWidgets(Composite parent)
 	{
-		instPreview = new InstructionTable(parent, new DisplaySettings(), getSite().getWorkbenchWindow().getWorkbench().getThemeManager());
+		instPreview = new InstructionTable(parent, new DisplaySettings(), getSite().getWorkbenchWindow().getWorkbench().getThemeManager(),
+				false);
 		instPreview.getTableViewer().getControl().setLayoutData(new GridData(SWT.FILL, SWT.FILL, true, false));
 		contentProvider = new ActiveInstructionPreviewContentProvider(instPreview.getTableViewer());
 		instPreview.setContentProvider(contentProvider);
@@ -183,8 +183,7 @@ public class SimulationView extends ViewPart
 
 	private void debugContextChanged(Optional<MachineDebugTarget> newTarget)
 	{
-		// if we didn't find a selected MachineDebugTarget
-		// call binToDebugTarget even if this.debugTarget==null
+		// call binToDebugTarget even if we didn't find a selected MachineDebugTarget
 		bindToDebugTarget(newTarget.orElse(null));
 	}