diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2017-04-11 18:25:04 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2017-04-11 18:25:04 +0200 |
| commit | 15c98f1740708ad7c05753ea28d5ad8ed4430e02 (patch) | |
| tree | fa58c228a546401099a108f95d62475643ae7c11 /jvmfwk | |
| parent | 583508ced26a422145111242c027b3fbd5687dc5 (diff) | |
jfw_existJRE requires non-null args, never returns JFW_E_INVALID_ARG
Change-Id: I2067b77393427a5a6a4273c2f609ff8d335f436a
Diffstat (limited to 'jvmfwk')
| -rw-r--r-- | jvmfwk/plugins/sunmajor/pluginlib/sunjavaplugin.cxx | 6 | ||||
| -rw-r--r-- | jvmfwk/source/framework.cxx | 3 |
2 files changed, 4 insertions, 5 deletions
diff --git a/jvmfwk/plugins/sunmajor/pluginlib/sunjavaplugin.cxx b/jvmfwk/plugins/sunmajor/pluginlib/sunjavaplugin.cxx index 860cbc4f2522..a5f11e6a3c21 100644 --- a/jvmfwk/plugins/sunmajor/pluginlib/sunjavaplugin.cxx +++ b/jvmfwk/plugins/sunmajor/pluginlib/sunjavaplugin.cxx @@ -30,6 +30,7 @@ #include <string.h> +#include <cassert> #include <memory> #include "config_options.h" #include "osl/diagnose.h" @@ -884,9 +885,10 @@ javaPluginError jfw_plugin_startJavaVirtualMachine( javaPluginError jfw_plugin_existJRE(const JavaInfo *pInfo, bool *exist) { + assert(pInfo != nullptr); + assert(exist != nullptr); + javaPluginError ret = javaPluginError::NONE; - if (!pInfo || !exist) - return javaPluginError::InvalidArg; OUString sLocation(pInfo->sLocation); if (sLocation.isEmpty()) diff --git a/jvmfwk/source/framework.cxx b/jvmfwk/source/framework.cxx index c094ea6a8da1..a0208180dc30 100644 --- a/jvmfwk/source/framework.cxx +++ b/jvmfwk/source/framework.cxx @@ -987,9 +987,6 @@ javaFrameworkError jfw_existJRE(const JavaInfo *pInfo, bool *exist) case javaPluginError::NONE: ret = JFW_E_NONE; break; - case javaPluginError::InvalidArg: - ret = JFW_E_INVALID_ARG; - break; case javaPluginError::Error: ret = JFW_E_ERROR; break; |