From 77e5a6d857bd3a1a6a59891aab9b5d9bb9bc85cf Mon Sep 17 00:00:00 2001 From: Stephan Bergmann Date: Thu, 13 Apr 2017 12:27:50 +0200 Subject: Replace a use of CJavaInfo with std::unique_ptr Change-Id: Ib659d6edf362901a95cfb50ac008520805ad0fb0 --- jvmfwk/source/framework.cxx | 51 ++++++++++++++++++++------------------------- jvmfwk/source/framework.hxx | 2 -- 2 files changed, 23 insertions(+), 30 deletions(-) (limited to 'jvmfwk') diff --git a/jvmfwk/source/framework.cxx b/jvmfwk/source/framework.cxx index 4786b9424844..d817bd36ee2c 100644 --- a/jvmfwk/source/framework.cxx +++ b/jvmfwk/source/framework.cxx @@ -366,7 +366,7 @@ javaFrameworkError jfw_findAndSelectJRE(std::unique_ptr *pInfo) if (jfw::getMode() == jfw::JFW_MODE_DIRECT) return JFW_E_DIRECT_MODE; sal_uInt64 nFeatureFlags = 0; - jfw::CJavaInfo aCurrentInfo; + std::unique_ptr aCurrentInfo; //Determine if accessibility support is needed bool bSupportAccessibility = jfw::isAccessibilitySupportDesired(); nFeatureFlags = bSupportAccessibility ? @@ -403,7 +403,7 @@ javaFrameworkError jfw_findAndSelectJRE(std::unique_ptr *pInfo) if (jfw_plugin_getJavaInfoFromJavaHome(versionInfos, &pHomeInfo, infos) == javaPluginError::NONE) { - aCurrentInfo = pHomeInfo; + aCurrentInfo.reset(pHomeInfo); // compare features // if the user does not require any features (nFeatureFlags = 0) @@ -433,19 +433,19 @@ javaFrameworkError jfw_findAndSelectJRE(std::unique_ptr *pInfo) // if the current Java installation implements all required features: use it if ((pJInfo->nFeatures & nFeatureFlags) == nFeatureFlags) { - aCurrentInfo = pJInfo; + aCurrentInfo.reset(pJInfo); bInfoFound = true; } - else if (static_cast(aCurrentInfo) == nullptr) + else if (!aCurrentInfo) { // current Java installation does not provide all features // but no Java installation has been detected before // -> remember the current one until one is found // that provides all features - aCurrentInfo = pJInfo; + aCurrentInfo.reset(pJInfo); } - - delete pJInfo; + else + delete pJInfo; } ++it; } @@ -494,10 +494,6 @@ javaFrameworkError jfw_findAndSelectJRE(std::unique_ptr *pInfo) { JavaInfo* pJInfo = arInfos[ii]; - //We remember the first installation in aCurrentInfo - // if no JavaInfo has been found before - if (aCurrentInfo.getLocation().isEmpty()) - aCurrentInfo = pJInfo; // compare features // If the user does not require any features (nFeatureFlags = 0) // then the first installation is used @@ -505,10 +501,18 @@ javaFrameworkError jfw_findAndSelectJRE(std::unique_ptr *pInfo) { //the just found Java implements all required features //currently there is only accessibility!!! - aCurrentInfo = pJInfo; + aCurrentInfo.reset( + jfw::CJavaInfo::copyJavaInfo(pJInfo)); bInfoFound = true; break; } + else if (!aCurrentInfo) + { + // We remember the first installation in aCurrentInfo if + // no JavaInfo has been found before: + aCurrentInfo.reset( + jfw::CJavaInfo::copyJavaInfo(pJInfo)); + } } //The array returned by jfw_plugin_getAllJavaInfos must be freed as well as //its contents @@ -521,7 +525,7 @@ javaFrameworkError jfw_findAndSelectJRE(std::unique_ptr *pInfo) //All Java installations found by the current plug-in lib //do not provide the required features. Try the next plug-in } - if (static_cast(aCurrentInfo) == nullptr) + if (!aCurrentInfo) {//The plug-ins did not find a suitable Java. Now try the paths which have been //added manually. //get the list of paths to jre locations which have been added manually @@ -565,15 +569,15 @@ javaFrameworkError jfw_findAndSelectJRE(std::unique_ptr *pInfo) { //the just found Java implements all required features //currently there is only accessibility!!! - aCurrentInfo = aInfo.release(); + aCurrentInfo = std::move(aInfo); bInfoFound = true; break; } - else if (aCurrentInfo.getLocation().isEmpty()) + else if (!aCurrentInfo) { // We remember the very first installation in // aCurrentInfo: - aCurrentInfo = aInfo.release(); + aCurrentInfo = std::move(aInfo); } } }//end iterate over paths @@ -582,18 +586,17 @@ javaFrameworkError jfw_findAndSelectJRE(std::unique_ptr *pInfo) }// end iterate plug-ins } } - if (static_cast(aCurrentInfo)) + if (aCurrentInfo) { jfw::NodeJava javaNode(jfw::NodeJava::USER); - javaNode.setJavaInfo(aCurrentInfo,true); + javaNode.setJavaInfo(aCurrentInfo.get(),true); javaNode.write(); //remember that this JRE was selected in this process jfw::setJavaSelected(); if (pInfo !=nullptr) { - //copy to out param - pInfo->reset(jfw::CJavaInfo::copyJavaInfo(aCurrentInfo.pInfo)); + *pInfo = std::move(aCurrentInfo); } } else @@ -1036,14 +1039,6 @@ CJavaInfo & CJavaInfo::operator = (const ::JavaInfo* info) return *this; } -OUString CJavaInfo::getLocation() const -{ - if (pInfo) - return OUString(pInfo->sLocation); - else - return OUString(); -} - } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/jvmfwk/source/framework.hxx b/jvmfwk/source/framework.hxx index 04fdf86165bf..3f9e5ff6ab85 100644 --- a/jvmfwk/source/framework.hxx +++ b/jvmfwk/source/framework.hxx @@ -57,8 +57,6 @@ public: const ::JavaInfo* operator ->() const { return pInfo;} operator ::JavaInfo* () { return pInfo;} - OUString getLocation() const; - static JavaInfo * copyJavaInfo(const JavaInfo * pInfo); }; -- cgit v1.2.3