Merge pull request #5 from apache/bug/SLING-9392-fixing-null-pointer

Fixing git SLING-9392 - Supplying default values for all array parameters
diff --git a/src/main/java/org/apache/sling/feature/modelconverter/Main.java b/src/main/java/org/apache/sling/feature/modelconverter/Main.java
index fbbee9a..e2582c7 100644
--- a/src/main/java/org/apache/sling/feature/modelconverter/Main.java
+++ b/src/main/java/org/apache/sling/feature/modelconverter/Main.java
@@ -65,16 +65,16 @@
     private String name;
 
     @Option(names = { "-d", "--dropVariable" }, description = "Variable (by name) in a Feature Model to be excluded (repeat for more)")
-    private List<String> dropVariables;
+    private List<String> dropVariables = new ArrayList<>();
 
     @Option(names = { "-a", "--addFrameworkProperty" }, description = "Adds Framework Property to Feature Models. Format: <Model Name>:<Property Name>=<value> (repeat for more)")
-    private List<String> addFrameworkProperties;
+    private List<String> addFrameworkProperties = new ArrayList<>();
 
     @Option(names = { "-D", "--noProvisioningModelName" }, description = "If flagged then the Provisioning Model Name is not added")
     private boolean noProvisioningModelName;
 
     @Option(names = { "-e", "--excludeBundle" }, description = "Bundle and/or Bundle Configuration to be excluded (repeat for more)")
-    private List<String> excludeBundles;
+    private List<String> excludeBundles = new ArrayList<>();
 
     @Option(names = { "-r", "--runMode" }, description = "Runmode to add to this build (all no-runmodes are included by default, repeat for more)")
     private List<String> runModes;
@@ -122,6 +122,7 @@
                     modelMap.put(propName, propValue);
                 }
             }
+            
             options.put("addFrameworkProperties", frameworkPropertiesMap);
             options.put("excludeBundles", excludeBundles);
             LOGGER.info("Excluded Bundles: '{}'", excludeBundles);