Moved TODOs from getting_started.md to the source code

[Mograsim.git] / plugins / net.mograsim.plugin.core / src / net / mograsim / plugin / views / SimulationView.java

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 9f9f60b..f81fb4b 100644 (file)
--- 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));
        }