launch option: write additional variable options
authorjihye424.kim <jihye424.kim@samsung.com>
Thu, 24 Dec 2015 08:01:24 +0000 (17:01 +0900)
committerjihye424.kim <jihye424.kim@samsung.com>
Thu, 24 Dec 2015 08:01:24 +0000 (17:01 +0900)
- plugin add new variable (string type)
-- old: do not write this variable to launch config file
-- new: write this variable to launch config file

Change-Id: I053cb6262655c30aa5e40809db35b297c1312a68
Signed-off-by: jihye424.kim <jihye424.kim@samsung.com>
src/org/tizen/emulator/manager/vms/option/LaunchConfig.java

index 9bbe6ea..43fb63c 100644 (file)
@@ -83,10 +83,7 @@ public class LaunchConfig {
        }
 
        public void addVariable(OPTION_KEY key, String value, String comment) {
-               if (comment != null && !comment.isEmpty()) {
-                       varComment.put(key.toString(), comment);
-               }
-               addVariable(key.toString(), value);
+               addVariable(key.toString(), value, comment);
        }
 
        public void addQemuOption(String option) {
@@ -165,8 +162,22 @@ public class LaunchConfig {
                                if ((var = variables.get(key)) != null) {
                                        bw.write(key.toString() + "=" + var); //$NON-NLS-1$
                                        bw.newLine();
+                                       // delete key from variables map
+                                       variables.remove(key);
+                               }
+                       }
+                       // write additional variables
+                       for (String key : variables.keySet()) {
+                               if ((comment = varComment.get(key)) != null) {
+                                       bw.write(comment);
+                                       bw.newLine();
+                               }
+                               if ((var = variables.get(key)) != null) {
+                                       bw.write(key.toString() + "=" + var); //$NON-NLS-1$
+                                       bw.newLine();
                                }
                        }
+
                        String kernelOption = ""; //$NON-NLS-1$
                        for (String str : kernelOptions) {
                                if (!str.isEmpty()) {