diff options
-rw-r--r-- | jvmfwk/plugins/sunmajor/pluginlib/sunjavaplugin.cxx | 23 | ||||
-rw-r--r-- | jvmfwk/plugins/sunmajor/pluginlib/sunversion.cxx | 4 | ||||
-rw-r--r-- | jvmfwk/plugins/sunmajor/pluginlib/util.cxx | 2 | ||||
-rw-r--r-- | jvmfwk/source/elements.cxx | 2 | ||||
-rw-r--r-- | jvmfwk/source/framework.cxx | 4 |
5 files changed, 21 insertions, 14 deletions
diff --git a/jvmfwk/plugins/sunmajor/pluginlib/sunjavaplugin.cxx b/jvmfwk/plugins/sunmajor/pluginlib/sunjavaplugin.cxx index aa4091c7c991..481847e1cdfc 100644 --- a/jvmfwk/plugins/sunmajor/pluginlib/sunjavaplugin.cxx +++ b/jvmfwk/plugins/sunmajor/pluginlib/sunjavaplugin.cxx @@ -254,7 +254,8 @@ javaPluginError jfw_plugin_getAllJavaInfos( { //The minVersion was not recognized as valid for this vendor. JFW_ENSURE( - 0,"[Java framework]sunjavaplugin does not know version: " + false, + "[Java framework]sunjavaplugin does not know version: " + ouMinVer + " for vendor: " + cur->getVendor() + " .Check minimum Version." ); return JFW_PLUGIN_E_WRONG_VERSION_FORMAT; @@ -272,7 +273,8 @@ javaPluginError jfw_plugin_getAllJavaInfos( { //The maxVersion was not recognized as valid for this vendor. JFW_ENSURE( - 0,"[Java framework]sunjavaplugin does not know version: " + false, + "[Java framework]sunjavaplugin does not know version: " + ouMaxVer + " for vendor: " + cur->getVendor() + " .Check maximum Version." ); return JFW_PLUGIN_E_WRONG_VERSION_FORMAT; @@ -295,7 +297,8 @@ javaPluginError jfw_plugin_getAllJavaInfos( { //The excluded version was not recognized as valid for this vendor. JFW_ENSURE( - 0,"[Java framework]sunjavaplugin does not know version: " + false, + "[Java framework]sunjavaplugin does not know version: " + sExVer + " for vendor: " + cur->getVendor() + " .Check excluded versions." ); return JFW_PLUGIN_E_WRONG_VERSION_FORMAT; @@ -378,7 +381,8 @@ javaPluginError jfw_plugin_getJavaInfoByPath( { //The minVersion was not recognized as valid for this vendor. JFW_ENSURE( - 0,"[Java framework]sunjavaplugin does not know version: " + false, + "[Java framework]sunjavaplugin does not know version: " + ouMinVer + " for vendor: " + aVendorInfo->getVendor() + " .Check minimum Version." ); return JFW_PLUGIN_E_WRONG_VERSION_FORMAT; @@ -398,7 +402,8 @@ javaPluginError jfw_plugin_getJavaInfoByPath( { //The maxVersion was not recognized as valid for this vendor. JFW_ENSURE( - 0,"[Java framework]sunjavaplugin does not know version: " + false, + "[Java framework]sunjavaplugin does not know version: " + ouMaxVer + " for vendor: " + aVendorInfo->getVendor() + " .Check maximum Version." ); return JFW_PLUGIN_E_WRONG_VERSION_FORMAT; @@ -419,7 +424,8 @@ javaPluginError jfw_plugin_getJavaInfoByPath( { //The excluded version was not recognized as valid for this vendor. JFW_ENSURE( - 0,"[Java framework]sunjavaplugin does not know version: " + false, + "[Java framework]sunjavaplugin does not know version: " + sExVer + " for vendor: " + aVendorInfo->getVendor() + " .Check excluded versions." ); return JFW_PLUGIN_E_WRONG_VERSION_FORMAT; @@ -579,7 +585,8 @@ javaPluginError jfw_plugin_startJavaVirtualMachine( if ((moduleRt = osl_loadModule(sRuntimeLib.pData, SAL_LOADMODULE_DEFAULT)) == 0) #endif { - JFW_ENSURE(0, "[Java framework]sunjavaplugin" SAL_DLLEXTENSION + JFW_ENSURE(false, + "[Java framework]sunjavaplugin" SAL_DLLEXTENSION " could not load Java runtime library: \n" + sRuntimeLib + "\n"); JFW_TRACE0("[Java framework]sunjavaplugin" SAL_DLLEXTENSION @@ -606,7 +613,7 @@ javaPluginError jfw_plugin_startJavaVirtualMachine( moduleRt, sSymbolCreateJava.pData); if (!pCreateJavaVM) { - OSL_ASSERT(0); + OSL_ASSERT(false); OString sLib = OUStringToOString( sRuntimeLib, osl_getThreadTextEncoding()); OString sSymbol = OUStringToOString( diff --git a/jvmfwk/plugins/sunmajor/pluginlib/sunversion.cxx b/jvmfwk/plugins/sunmajor/pluginlib/sunversion.cxx index 2cca0c2831a4..9e3114304aca 100644 --- a/jvmfwk/plugins/sunmajor/pluginlib/sunversion.cxx +++ b/jvmfwk/plugins/sunmajor/pluginlib/sunversion.cxx @@ -74,7 +74,7 @@ bool SunVersion::init(const char *szVersion) char buf[128]; //char must me a number 0 - 999 and no leading - while (1) + while (true) { if (pCur < pEnd && isdigit(*pCur)) { @@ -128,7 +128,7 @@ bool SunVersion::init(const char *szVersion) if (* (pCur - 1) == '_') {// _01, _02 // update is the last part _01, _01a, part 0 is the digits parts and 1 the trailing alpha - while (1) + while (true) { if (pCur <= pEnd) { diff --git a/jvmfwk/plugins/sunmajor/pluginlib/util.cxx b/jvmfwk/plugins/sunmajor/pluginlib/util.cxx index 238842d0578f..f87aa7221e9f 100644 --- a/jvmfwk/plugins/sunmajor/pluginlib/util.cxx +++ b/jvmfwk/plugins/sunmajor/pluginlib/util.cxx @@ -460,7 +460,7 @@ bool getJavaProps(const OUString & exePath, //Use this thread to read output stream FileHandleReader::Result rs = FileHandleReader::RESULT_OK; - while (1) + while (true) { OString aLine; rs = stdoutReader.readLine( & aLine); diff --git a/jvmfwk/source/elements.cxx b/jvmfwk/source/elements.cxx index db5c3e24bd22..9fdab7a70811 100644 --- a/jvmfwk/source/elements.cxx +++ b/jvmfwk/source/elements.cxx @@ -1074,7 +1074,7 @@ void MergedSettings::merge(const NodeJava & share, const NodeJava & user) else if (share.getEnabled()) m_bEnabled = * share.getEnabled(); else - m_bEnabled = sal_True; + m_bEnabled = true; if (user.getUserClassPath()) m_sClassPath = * user.getUserClassPath(); diff --git a/jvmfwk/source/framework.cxx b/jvmfwk/source/framework.cxx index 38109892a23d..f0b9d692b172 100644 --- a/jvmfwk/source/framework.cxx +++ b/jvmfwk/source/framework.cxx @@ -371,7 +371,7 @@ javaFrameworkError SAL_CALL jfw_startVM( "-Djava.class.path=" + jfw::BootParams::getClasspath(); } else - OSL_ASSERT(0); + OSL_ASSERT(false); pInfo = aInfo.pInfo; } assert(pInfo != NULL); @@ -884,7 +884,7 @@ javaFrameworkError SAL_CALL jfw_getJavaInfoByPath( {// plugin does not recognize this path as belonging to JRE continue; } - OSL_ASSERT(0); + OSL_ASSERT(false); } if (*ppInfo == NULL && errcode != JFW_E_FAILED_VERSION) errcode = JFW_E_NOT_RECOGNIZED; |