PluggableDiscovery: Factored out method to umarshal BoardPort from JSON

Author: cmaglie

arduino-core/src/cc/arduino/packages/BoardPort.java

@@ -126,7 +126,7 @@ public String toString() {
   // If found, boardName is set to the name from boards.txt
   // and the board is returned.  If not found, null is returned.
   public TargetBoard searchMatchingBoard() {
-    if (identificationPrefs.isEmpty()) return null;
+    if (identificationPrefs == null || identificationPrefs.isEmpty()) return null;
     for (TargetPackage targetPackage : BaseNoGui.packages.values()) {
       for (TargetPlatform targetPlatform : targetPackage.getPlatforms().values()) {
         for (TargetBoard board : targetPlatform.getBoards().values()) {

arduino-core/src/cc/arduino/packages/discoverers/PluggableDiscovery.java

@@ -141,37 +141,26 @@ private void processJsonNode(ObjectMapper mapper, JsonNode node) {
 
       portList.clear();
       portsNode.forEach(portNode -> {
-        try {
-          BoardPort port = mapper.treeToValue(portNode, BoardPort.class);
-          port.searchMatchingBoard();
+        BoardPort port = mapJsonNodeToBoardPort(mapper, node);
+        if (port != null) {
           addOrUpdate(port);
-        } catch (JsonProcessingException e) {
-          System.err.println(format("{0}: Invalid BoardPort message", discoveryName));
-          e.printStackTrace();
         }
       });
       return;
 
     // Messages for SYNC updates
 
     case "add":
-      try {
-        BoardPort port = mapper.treeToValue(node.get("port"), BoardPort.class);
-        port.searchMatchingBoard();
-        addOrUpdate(port);
-      } catch (JsonProcessingException e) {
-        System.err.println(format("{0}: Invalid BoardPort message", discoveryName));
-        e.printStackTrace();
+      BoardPort addedPort = mapJsonNodeToBoardPort(mapper, node);
+      if (addedPort != null) {
+        addOrUpdate(addedPort);
       }
       return;
 
     case "remove":
-      try {
-        BoardPort port = mapper.treeToValue(node.get("port"), BoardPort.class);
-        remove(port);
-      } catch (JsonProcessingException e) {
-        System.err.println(format("{0}: Invalid BoardPort message", discoveryName));
-        e.printStackTrace();
+      BoardPort removedPort = mapJsonNodeToBoardPort(mapper, node);
+      if (removedPort != null) {
+        remove(removedPort);
       }
       return;
 
@@ -181,6 +170,18 @@ private void processJsonNode(ObjectMapper mapper, JsonNode node) {
     }
   }
 
+  private BoardPort mapJsonNodeToBoardPort(ObjectMapper mapper, JsonNode node) {
+    try {
+      BoardPort port = mapper.treeToValue(node.get("port"), BoardPort.class);
+      port.searchMatchingBoard();
+      return port;
+    } catch (JsonProcessingException e) {
+      System.err.println(format("{0}: Invalid BoardPort message", discoveryName));
+      e.printStackTrace();
+      return null;
+    }
+  }
+
   @Override
   public void start() throws Exception {
     try {