Fix DeviceActivity#persistableAddresses to be more graceful (fixes #147)

and accept the same address syntax as syncthing core web UI does.

app/src/main/java/com/nutomic/syncthingandroid/activities/DeviceActivity.java

@@ -530,9 +530,27 @@ private void updateDevice() {
     }
 
     private List<String> persistableAddresses(CharSequence userInput) {
-        return isEmpty(userInput)
-                ? DYNAMIC_ADDRESS
-                : Arrays.asList(userInput.toString().split(" "));
+        if (isEmpty(userInput)) {
+            return DYNAMIC_ADDRESS;
+        }
+
+        /**
+         * Be fault-tolerant here.
+         * The user can write like this:
+         * tcp4://192.168.1.67:2222, dynamic
+         * tcp4://192.168.1.67:2222; dynamic
+         * tcp4://192.168.1.67:2222,dynamic
+         * tcp4://192.168.1.67:2222;dynamic
+         * tcp4://192.168.1.67:2222 dynamic
+         */
+        String input = userInput.toString();
+        input = input.replace(",", " ");
+        input = input.replace(";", " ");
+        input = input.replaceAll("\\s+", ", ");
+        // Log.v(TAG, "persistableAddresses: Cleaned user input=" + input);
+
+        // Split and return the addresses as String[].
+        return Arrays.asList(input.split(", "));
     }
 
     private String displayableAddresses() {
@@ -542,7 +560,7 @@ private String displayableAddresses() {
         List<String> list = DYNAMIC_ADDRESS.equals(mDevice.addresses)
                 ? DYNAMIC_ADDRESS
                 : mDevice.addresses;
-        return TextUtils.join(" ", list);
+        return TextUtils.join(", ", list);
     }
 
     @Override