diff -r 88e2753a2334 -r 0fe62353385b src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/MacAppStoreBundler.java --- a/src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/MacAppStoreBundler.java Wed Oct 23 10:10:34 2019 -0400 +++ b/src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/MacAppStoreBundler.java Wed Oct 23 10:37:54 2019 -0400 @@ -28,11 +28,7 @@ import java.io.File; import java.io.IOException; import java.text.MessageFormat; -import java.util.ArrayList; -import java.util.List; -import java.util.Map; -import java.util.Optional; -import java.util.ResourceBundle; +import java.util.*; import static jdk.jpackage.internal.StandardBundlerParam.*; import static jdk.jpackage.internal.MacAppBundler.*; @@ -235,11 +231,7 @@ public boolean validate(Map params) throws ConfigException { try { - if (params == null) { - throw new ConfigException( - I18N.getString("error.parameters-null"), - I18N.getString("error.parameters-null.advice")); - } + Objects.requireNonNull(params); // hdiutil is always available so there's no need to test for // availability.