From 9735731c596cb2809dc4f1e69f37443fab3befaf Mon Sep 17 00:00:00 2001
From: Joachim Klein <klein@tcs.inf.tu-dresden.de>
Date: Fri, 21 Jul 2017 16:21:26 +0000
Subject: [PATCH] PrismSettings: refactor option splitting (for
 -switch:options), provide access to raw option string

git-svn-id: https://www.prismmodelchecker.org/svn/prism/prism/trunk@12136 bbc10eb1-c90d-0410-af57-cb519fbb1720
---
 prism/src/prism/PrismSettings.java | 50 +++++++++++++++++++++---------
 1 file changed, 35 insertions(+), 15 deletions(-)

diff --git a/prism/src/prism/PrismSettings.java b/prism/src/prism/PrismSettings.java
index d598803e..b4ade202 100644
--- a/prism/src/prism/PrismSettings.java
+++ b/prism/src/prism/PrismSettings.java
@@ -898,9 +898,10 @@ public class PrismSettings implements Observer
 		double d;
 		
 		// Process string (remove - and extract any options) 
-		Pair<String, Map<String, String>> pair = splitSwitch(args[i]);
+		Pair<String, String> pair = splitSwitch(args[i]);
 		String sw = pair.first;
-		Map<String, String> options = pair.second; 
+		String optionsString = pair.second;
+		Map<String, String> options = splitOptionsString(optionsString);
 		
 		// Note: the order of these switches should match the -help output (just to help keep track of things).
 		
@@ -1602,9 +1603,9 @@ public class PrismSettings implements Observer
 	 * When present, the options is a comma-separated list of "option" or "option=value" items.
 	 * The switch itself can be prefixed with either 1 or 2 hyphens.
 	 * 
-	 * @return a pair containing the switch name and a mapping from options to values.  
+	 * @return a pair containing the switch name and the (optional, may be null) options part
 	 */
-	private static Pair<String, Map<String, String>> splitSwitch(String sw) throws PrismException
+	private static Pair<String, String> splitSwitch(String sw)
 	{
 		// Remove "-"
 		sw = sw.substring(1);
@@ -1613,21 +1614,40 @@ public class PrismSettings implements Observer
 			sw = sw.substring(1);
 		// Extract options, if present
 		int i = sw.indexOf(':');
-		Map<String,String> map = new HashMap<String, String>();
+
+		String optionsString = null;
 		if (i != -1) {
-			String optionsString = sw.substring(i + 1);
+			optionsString = sw.substring(i + 1);
 			sw = sw.substring(0, i);
-			String options[] = optionsString.split(",");
-			for (String option : options) {
-				int j = option.indexOf("=");
-				if (j == -1) {
-					map.put(option, null);
-				} else {
-					map.put(option.substring(0,j), option.substring(j+1));
-				}
+		}
+
+		return new Pair<String, String>(sw, optionsString);
+	}
+
+	/**
+	 * Split an options string (see splitSwitch)
+	 * into a map from options to values.
+	 * <br>
+	 * For "option" options, the value is {@code null}.
+	 * @return a mapping from options to values.
+	 */
+	private static Map<String, String> splitOptionsString(String optionsString)
+	{
+		Map<String,String> map = new HashMap<String, String>();
+		if (optionsString == null || "".equals(optionsString))
+			return map;
+
+		String options[] = optionsString.split(",");
+		for (String option : options) {
+			int j = option.indexOf("=");
+			if (j == -1) {
+				map.put(option, null);
+			} else {
+				map.put(option.substring(0,j), option.substring(j+1));
 			}
 		}
-		return new Pair<String, Map<String,String>>(sw, map);
+
+		return map;
 	}
 
 	/**