Add debugging tweaks for mobility models

Javadoc and comments might follow later

src/gui/DTNSimGUI.java

@@ -257,6 +257,10 @@ public class DTNSimGUI extends DTNSimUI {
 		infoPanel.showInfo(host);
 	}
 
+	public void highlightHost(DTNHost host) {
+		field.highlightHost(host);
+	}
+
 	/**
 	 * Shows a path on the playfield
 	 * @param path The path to show

src/gui/EventLogPanel.java

@@ -252,6 +252,7 @@ public class EventLogPanel extends JPanel
 			gui.setPaused(true);
 			if (host1 != null) {
 				gui.setFocus(host1);
+				gui.highlightHost(host1);
 			}
 		}
 
@@ -316,6 +317,7 @@ public class EventLogPanel extends JPanel
 		if (source.getClientProperty(HOST_PROP) != null) {
 			// button was a host button -> focus it on GUI
 			gui.setFocus((DTNHost)source.getClientProperty(HOST_PROP));
+			gui.highlightHost((DTNHost)source.getClientProperty(HOST_PROP));
 		}
 		else if (source.getClientProperty(MSG_PROP) != null) {
 			// was a message button -> show information about the message

src/gui/MainWindow.java

@@ -73,6 +73,7 @@ public class MainWindow extends JFrame {
         hostListScroll = new JScrollPane(chooser);
         hostListScroll.setHorizontalScrollBarPolicy(
 		JScrollPane.HORIZONTAL_SCROLLBAR_NEVER);
+        hostListScroll.getVerticalScrollBar().setUnitIncrement(20);
 
         logControlSplit = new JSplitPane(JSplitPane.HORIZONTAL_SPLIT,
 		new JScrollPane(elp.getControls()),new JScrollPane(elp));

src/gui/NodeChooser.java

@@ -200,6 +200,7 @@ public class NodeChooser extends JPanel implements ActionListener {
 			JButton source = (JButton)e.getSource();
 			DTNHost host = (DTNHost)source.getClientProperty(HOST_KEY);
 			gui.setFocus(host);
+			gui.highlightHost(host);
 		}
 		else if (e.getSource() == this.groupChooser) {
 			setNodes(groupChooser.getSelectedIndex() * MAX_NODE_COUNT);

src/gui/SimMenuBar.java

@@ -121,7 +121,7 @@ public class SimMenuBar extends JMenuBar implements ActionListener {
 		showBuffer = createCheckItem(pfMenu,
 				"Show message buffer", true, SHOW_BUFFER_S);
 		focusOnClick = createCheckItem(pfMenu,
-				"Focus to closest node on mouse click", false,FOCUS_ON_CLICK_S);
+				"Focus to closest node on mouse click", true,FOCUS_ON_CLICK_S);
 
 		enableMapGraphic = createCheckItem(pfMenu,"Show map graphic",
 				true, null);

src/gui/playfield/PlayField.java

@@ -341,12 +341,16 @@ public class PlayField extends JPanel implements VhmListener {
                 closestDist = dist;
             }
         }
-        highlightedHosts.clear();
-        highlightedHosts.add(closest);
-        NodeGraphic.setHighlightedNodes(highlightedHosts);
 
         gui.setFocus(closest);
 
+        highlightHost(closest);
+    }
+
+    public void highlightHost(DTNHost host) {
+        highlightedHosts.clear();
+        highlightedHosts.add(host);
+        NodeGraphic.setHighlightedNodes(highlightedHosts);
         this.updateField();
     }
 

[Ghost User]

SonarQube analysis reported 5 issues

• ⚠ 4 major
• 🔽 1 minor

1. ⚠ Remove this unused private "addEvent" method. 📘
2. ⚠ Remove this unused private "addInfoButton" method. 📘
3. ⚠ Provide multiple methods instead of using "firstDelivery" to determine which action to take. 📘
4. ⚠ Assign this magic number 20 to a well-named constant, and use the constant instead. 📘
5. 🔽 Make this line start at column 9. 📘