Non-default values are now highlighted in the InstructionTable

[Mograsim.git] / plugins / net.mograsim.plugin.core / src / net / mograsim / plugin / tables / mi / InstructionView.java

diff --git a/plugins/net.mograsim.plugin.core/src/net/mograsim/plugin/tables/mi/InstructionView.java b/plugins/net.mograsim.plugin.core/src/net/mograsim/plugin/tables/mi/InstructionView.java
index b352e09..0158c7b 100644 (file)
--- a/plugins/net.mograsim.plugin.core/src/net/mograsim/plugin/tables/mi/InstructionView.java
+++ b/plugins/net.mograsim.plugin.core/src/net/mograsim/plugin/tables/mi/InstructionView.java
@@ -19,6 +19,7 @@ import org.eclipse.ui.PartInitException;
 import org.eclipse.ui.part.EditorPart;
 
 import net.mograsim.machine.Memory.MemoryCellModifiedListener;
+import net.mograsim.machine.mi.AssignableMicroInstructionMemory.MIMemoryReassignedListener;
 import net.mograsim.machine.mi.MicroInstructionMemory;
 import net.mograsim.machine.mi.MicroInstructionMemory.ActiveMicroInstructionChangedListener;
 import net.mograsim.machine.mi.MicroInstructionMemoryParseException;
@@ -50,8 +51,7 @@ public class InstructionView extends EditorPart implements MemoryCellModifiedLis
                new RadixSelector(parent, displaySettings);
 
                addActivationButton(parent);
-
-               table = new InstructionTable(parent, displaySettings);
+               table = new InstructionTable(parent, displaySettings, getSite().getWorkbenchWindow().getWorkbench().getThemeManager());
                table.setContentProvider(provider);
                table.bindMicroInstructionMemory(memory);
 
@@ -67,9 +67,13 @@ public class InstructionView extends EditorPart implements MemoryCellModifiedLis
                        LazyTableViewer viewer = table.getTableViewer();
                        viewer.highlightRow(highlighted, false);
                        highlighted = index;
-                       viewer.highlightRow(index, true);
-                       viewer.getTable().showItem(viewer.getTable().getItem(Math.min((int) memory.getDefinition().getMaximalAddress(), index + 2)));
-                       viewer.getTable().showItem(viewer.getTable().getItem(index));
+                       if (index != -1)
+                       {
+                               viewer.highlightRow(index, true);
+                               viewer.getTable()
+                                               .showItem(viewer.getTable().getItem(Math.min((int) memory.getDefinition().getMaximalAddress(), index + 2)));
+                               viewer.getTable().showItem(viewer.getTable().getItem(index));
+                       }
                });
        }
 
@@ -77,8 +81,26 @@ public class InstructionView extends EditorPart implements MemoryCellModifiedLis
        {
                Button activationButton = new Button(parent, SWT.PUSH);
                activationButton.setText("Set Active");
-               activationButton.addListener(SWT.Selection,
-                               e -> context.getActiveMachine().ifPresent(m -> m.getMicroInstructionMemory().bind(memory)));
+               activationButton.addListener(SWT.Selection, e -> context.getActiveMachine().ifPresent(m ->
+               {
+                       // clear highlighting if the memory is reassigned
+                       MIMemoryReassignedListener memReassignedListener = n ->
+                       {
+                               if (n != memory)
+                                       highlight(-1);
+                       };
+                       m.getMicroInstructionMemory().registerMemoryReassignedListener(memReassignedListener);
+                       // clear highlighting if the active machine changes
+                       context.addActiveMachineListener(n ->
+                       {
+                               if (n.isEmpty() || n.get() != m)
+                               {
+                                       highlight(-1);
+                                       m.getMicroInstructionMemory().deregisterMemoryReassignedListener(memReassignedListener);
+                               }
+                       });
+                       m.getMicroInstructionMemory().bind(memory);
+               }));
        }
 
        public void bindMicroInstructionMemory(MicroInstructionMemory memory)
@@ -175,6 +197,7 @@ public class InstructionView extends EditorPart implements MemoryCellModifiedLis
                        {
                                IFileEditorInput fileInput = (IFileEditorInput) input;
                                context = ProjectMachineContext.getMachineContextOf(fileInput.getFile().getProject());
+                               context.activateMachine();
                                setPartName(fileInput.getName());
                                open(fileInput.getFile());
                        }
@@ -216,4 +239,11 @@ public class InstructionView extends EditorPart implements MemoryCellModifiedLis
        {
                highlight((int) (address - memory.getDefinition().getMinimalAddress()));
        }
+
+       @Override
+       public void dispose()
+       {
+               table.dispose();
+               super.dispose();
+       }
 }