Bug fix: Handle selection state with respect to tools properly.

Specifically when creating, editing or selecting links modules should be deselected at the start of the change not at the end (the latter is the current, confusing, indirect behaviour).

Author: EdNutting

src/com/modsim/gui/view/ViewUtil.java

@@ -277,6 +277,8 @@ public void mousePressed(MouseEvent e) {
                 else {
                     Port p = screenSpace_portAt(e.getX(), e.getY());
 
+                    Main.selection.clear();
+
                     //Link behaviour
                     if (p != null) {
                         tool = new MakeLinkTool();
@@ -408,7 +410,8 @@ public void keyPressed(KeyEvent e) {
             }
         }
         // Delete selection
-        else if (e.getKeyCode() == KeyEvent.VK_DELETE) {
+        else if (e.getKeyCode() == KeyEvent.VK_DELETE
+                 && !Main.selection.isEmpty()) {
             Main.selection.deleteAll();
         }
         // Pass to tool

src/com/modsim/tools/MakeLinkTool.java

@@ -94,7 +94,9 @@ public void paintWorld(Graphics2D g) {
 	public boolean startLink(int x, int y) {
 		source = ViewUtil.screenSpace_portAt(x, y);
 		if (source != null) {
-			working = true;
+            Main.selection.clear();
+
+            working = true;
 			start = new Vec2(x, y);
 			curve = new BezierPath();