Fixed GUIMerger / GUISplitter bit order
[Mograsim.git] / net.mograsim.logic.model / src / net / mograsim / logic / model / model / components / atomic / GUIMerger.java
index 287e437..5f0fe3a 100644 (file)
@@ -1,5 +1,7 @@
 package net.mograsim.logic.model.model.components.atomic;\r
 \r
+import org.eclipse.swt.SWT;\r
+\r
 import com.google.gson.JsonElement;\r
 import com.google.gson.JsonPrimitive;\r
 \r
@@ -20,12 +22,12 @@ import net.mograsim.preferences.Preferences;
 \r
 public class GUIMerger extends GUIComponent\r
 {\r
-       private static final double width = 20;\r
+       private static final double width = 10;\r
        private static final double heightPerPin = 10;\r
 \r
        public final int logicWidth;\r
 \r
-       private ReadEnd[] inputEnds;\r
+       private final ReadEnd[] inputEnds;\r
        private ReadEnd outputEnd;\r
 \r
        public GUIMerger(ViewModelModifiable model, int logicWidth, String name)\r
@@ -36,7 +38,8 @@ public class GUIMerger extends GUIComponent
                double inputHeight = 0;\r
                for (int i = 0; i < logicWidth; i++, inputHeight += 10)\r
                        addPin(new Pin(this, "I" + i, 1, 0, inputHeight));\r
-               addPin(new Pin(this, "O", logicWidth, width, logicWidth * heightPerPin / 2));\r
+               addPin(new Pin(this, "O", logicWidth, width, (logicWidth - 1) * heightPerPin / 2));\r
+               inputEnds = new ReadEnd[logicWidth];\r
        }\r
 \r
        @Override\r
@@ -45,20 +48,26 @@ public class GUIMerger extends GUIComponent
                double posX = getPosX();\r
                double posY = getPosY();\r
 \r
+               ColorDefinition c = BitVectorFormatter.formatAsColor(outputEnd);\r
+               if (c != null)\r
+                       gc.setForeground(ColorManager.current().toColor(c));\r
+               double outLineY = posY + (logicWidth - 1) * heightPerPin / 2;\r
+               gc.drawLine(posX + width / 2, outLineY, posX + width, outLineY);\r
                double inputHeight = posY;\r
                for (int i = 0; i < logicWidth; i++, inputHeight += 10)\r
                {\r
-                       ColorDefinition c = BitVectorFormatter.formatAsColor(inputEnds[i]);\r
+                       c = BitVectorFormatter.formatAsColor(inputEnds[i]);\r
                        if (c != null)\r
                                gc.setForeground(ColorManager.current().toColor(c));\r
                        gc.drawLine(posX, inputHeight, posX + width / 2, inputHeight);\r
                }\r
                gc.setForeground(Preferences.current().getColor("net.mograsim.logic.model.color.foreground"));\r
+               int oldLineCap = gc.getLineCap();\r
+               int lineJoin = gc.getLineJoin();\r
+               // TODO find better "replacement" for JOIN_BEVEL\r
+               gc.setLineCap(lineJoin == SWT.JOIN_MITER ? SWT.CAP_SQUARE : lineJoin == SWT.JOIN_ROUND ? SWT.CAP_ROUND : SWT.CAP_SQUARE);\r
                gc.drawLine(posX + width / 2, posY, posX + width / 2, posY + heightPerPin * (logicWidth - 1));\r
-               ColorDefinition c = BitVectorFormatter.formatAsColor(outputEnd);\r
-               if (c != null)\r
-                       gc.setForeground(ColorManager.current().toColor(c));\r
-               gc.drawLine(posX + width / 2, posY + heightPerPin * logicWidth / 2, posX + width, posY + heightPerPin * logicWidth / 2);\r
+               gc.setLineCap(oldLineCap);\r
        }\r
 \r
        @Override\r
@@ -69,7 +78,7 @@ public class GUIMerger extends GUIComponent
 \r
        public void setLogicModelBinding(ReadEnd[] inputEnds, ReadEnd outputEnd)\r
        {\r
-               this.inputEnds = inputEnds;\r
+               System.arraycopy(inputEnds, 0, this.inputEnds, 0, logicWidth);\r
                this.outputEnd = outputEnd;\r
        }\r
 \r