Invalid versions don't cause IDE to crash and exit any more. They are reported and contributions are considered missing version. Fixes #2926

Author: Federico Fissore

arduino-core/src/cc/arduino/contributions/VersionHelper.java

@@ -8,15 +8,20 @@ public static Version valueOf(String ver) {
     if (ver == null) {
       return null;
     }
-    String[] verParts = ver.split("\\.");
-    if (verParts.length < 3) {
-      if (verParts.length == 2) {
-        return Version.forIntegers(Integer.valueOf(verParts[0]), Integer.valueOf(verParts[1]));
+    try {
+      String[] verParts = ver.split("\\.");
+      if (verParts.length < 3) {
+        if (verParts.length == 2) {
+          return Version.forIntegers(Integer.valueOf(verParts[0]), Integer.valueOf(verParts[1]));
+        } else {
+          return Version.forIntegers(Integer.valueOf(verParts[0]));
+        }
       } else {
-        return Version.forIntegers(Integer.valueOf(verParts[0]));
+        return Version.valueOf(ver);
       }
-    } else {
-      return Version.valueOf(ver);
+    } catch (Exception e) {
+      System.err.println("Invalid version found: " + ver);
+      return null;
     }
   }
 

arduino-core/src/cc/arduino/contributions/libraries/ContributedLibrary.java

@@ -141,7 +141,7 @@ public boolean equals(Object obj) {
     String thisVersion = getParsedVersion();
     String otherVersion = ((ContributedLibrary) obj).getParsedVersion();
 
-    boolean versionEquals = thisVersion == null || otherVersion == null || thisVersion.equals(otherVersion);
+    boolean versionEquals = thisVersion == otherVersion || (thisVersion != null && otherVersion != null && thisVersion.equals(otherVersion));
 
     String thisName = getName();
     String otherName = ((ContributedLibrary) obj).getName();

arduino-core/src/cc/arduino/contributions/libraries/LibrariesIndex.java

@@ -48,6 +48,9 @@ public boolean apply(ContributedLibrary contributedLibrary) {
   }
 
   public ContributedLibrary find(String name, String version) {
+    if (name == null || version == null) {
+      return null;
+    }
     for (ContributedLibrary lib : find(name)) {
       if (version.equals(lib.getParsedVersion())) {
         return lib;

arduino-core/src/cc/arduino/contributions/packages/ContributedPackage.java

@@ -47,6 +47,9 @@ public abstract class ContributedPackage {
   public abstract List<ContributedTool> getTools();
 
   public ContributedPlatform findPlatform(String architecture, String version) {
+    if (architecture == null || version == null) {
+      return null;
+    }
     for (ContributedPlatform platform : getPlatforms()) {
       if (platform.getArchitecture().equals(architecture) && version.equals(platform.getParsedVersion()))
         return platform;