[Libreoffice-commits] core.git: sal/osl sal/qa sal/rtl
Libreoffice Gerrit user
logerrit at kemper.freedesktop.org
Thu Nov 22 11:08:20 UTC 2018
sal/osl/unx/profile.cxx | 14 +-------------
sal/osl/w32/module.cxx | 5 +----
sal/osl/w32/time.cxx | 12 ++++--------
sal/qa/osl/file/osl_File.cxx | 11 ++++++-----
sal/rtl/bootstrap.cxx | 17 ++++++++---------
sal/rtl/hash.cxx | 7 +------
6 files changed, 21 insertions(+), 45 deletions(-)
New commits:
commit 85acc270fc670443cd700c471d9b495e12d4e6dc
Author: Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Thu Nov 22 08:52:59 2018 +0200
Commit: Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Thu Nov 22 12:07:54 2018 +0100
improve function-local statics in sal
Change-Id: I0853cf13162bae44cf8a5c44a4546a73f05772d9
Reviewed-on: https://gerrit.libreoffice.org/63780
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/sal/osl/unx/profile.cxx b/sal/osl/unx/profile.cxx
index a986dfd0d41f..589662b9b62a 100644
--- a/sal/osl/unx/profile.cxx
+++ b/sal/osl/unx/profile.cxx
@@ -879,19 +879,7 @@ static osl_TStamp OslProfile_getFileStamp(osl_TFile* pFile)
static bool OslProfile_lockFile(const osl_TFile* pFile, osl_TLockMode eMode)
{
struct flock lock;
- /* boring hack, but initializers for static vars must be constant */
- static bool bIsInitialized = false;
- static bool bLockingDisabled;
-
- if ( !bIsInitialized )
- {
- sal_Char* pEnvValue;
- pEnvValue = getenv( "STAR_PROFILE_LOCKING_DISABLED" );
-
- bLockingDisabled = pEnvValue != nullptr;
-
- bIsInitialized = true;
- }
+ static bool const bLockingDisabled = getenv( "STAR_PROFILE_LOCKING_DISABLED" ) != nullptr;
if (pFile->m_Handle < 0)
{
diff --git a/sal/osl/w32/module.cxx b/sal/osl/w32/module.cxx
index de9ce4db421b..8de791a7a8e1 100644
--- a/sal/osl/w32/module.cxx
+++ b/sal/osl/w32/module.cxx
@@ -327,10 +327,7 @@ typedef BOOL (WINAPI *GetModuleInformation_PROC)(
static bool osl_addressGetModuleURL_NT_( void *pv, rtl_uString **pustrURL )
{
bool bSuccess = false; /* Assume failure */
- static HMODULE hModPsapi = nullptr;
-
- if ( !hModPsapi )
- hModPsapi = LoadLibraryW( L"PSAPI.DLL" );
+ static HMODULE hModPsapi = LoadLibraryW( L"PSAPI.DLL" );
if ( hModPsapi )
{
diff --git a/sal/osl/w32/time.cxx b/sal/osl/w32/time.cxx
index d1284b663fae..135aab368fc8 100644
--- a/sal/osl/w32/time.cxx
+++ b/sal/osl/w32/time.cxx
@@ -35,18 +35,14 @@ sal_Bool SAL_CALL osl_getSystemTime(TimeValue* pTimeVal)
typedef VOID (WINAPI *GetSystemTimePreciseAsFileTime_PROC)(LPFILETIME);
- static HMODULE hModule = nullptr;
- static GetSystemTimePreciseAsFileTime_PROC pGetSystemTimePreciseAsFileTime = nullptr;
-
OSL_ASSERT(pTimeVal != nullptr);
- if ( !hModule )
+ static GetSystemTimePreciseAsFileTime_PROC pGetSystemTimePreciseAsFileTime = [&]()
{
- hModule = GetModuleHandleW( L"Kernel32.dll" );
- if ( hModule )
- pGetSystemTimePreciseAsFileTime = reinterpret_cast<GetSystemTimePreciseAsFileTime_PROC>(
+ HMODULE hModule = GetModuleHandleW( L"Kernel32.dll" );
+ return reinterpret_cast<GetSystemTimePreciseAsFileTime_PROC>(
GetProcAddress(hModule, "GetSystemTimePreciseAsFileTime"));
- }
+ }();
// use ~1 microsecond resolution if available
if (pGetSystemTimePreciseAsFileTime)
diff --git a/sal/qa/osl/file/osl_File.cxx b/sal/qa/osl/file/osl_File.cxx
index fcc8c41cf4ab..ecb65ae8cd3f 100644
--- a/sal/qa/osl/file/osl_File.cxx
+++ b/sal/qa/osl/file/osl_File.cxx
@@ -4922,8 +4922,7 @@ namespace osl_Directory
static OUString const & get_test_path()
{
- static OUString test_path;
- if (test_path.isEmpty())
+ static OUString test_path = [&]()
{
OUString tmp;
osl::FileBase::RC rc = osl::FileBase::getTempDirURL(tmp);
@@ -4962,14 +4961,16 @@ namespace osl_Directory
#endif
tmp_x += OString(TEST_PATH_POSTFIX);
- rc = osl::FileBase::getFileURLFromSystemPath(OStringToOUString(tmp_x, RTL_TEXTENCODING_UTF8), test_path);
+ OUString tmpTestPath;
+ rc = osl::FileBase::getFileURLFromSystemPath(OStringToOUString(tmp_x, RTL_TEXTENCODING_UTF8), tmpTestPath);
CPPUNIT_ASSERT_EQUAL_MESSAGE
(
"Cannot convert the system path back to an URL",
osl::FileBase::E_None, rc
- );
- }
+ );
+ return tmpTestPath;
+ }();
return test_path;
}
diff --git a/sal/rtl/bootstrap.cxx b/sal/rtl/bootstrap.cxx
index 897de890d890..9c6675586ef9 100644
--- a/sal/rtl/bootstrap.cxx
+++ b/sal/rtl/bootstrap.cxx
@@ -155,10 +155,9 @@ static bool getFromCommandLineArgs(
{
OSL_ASSERT(value);
- static NameValueVector *pNameValueVector = nullptr;
- if (!pNameValueVector)
+ static NameValueVector nameValueVector = [&]()
{
- static NameValueVector nameValueVector;
+ NameValueVector tmp;
sal_Int32 nArgCount = osl_getCommandArgCount();
for(sal_Int32 i = 0; i < nArgCount; ++ i)
@@ -189,18 +188,18 @@ static bool getFromCommandLineArgs(
nameValue.sValue = nameValue.sValue.copy(0,nameValue.sValue.getLength()-1);
}
- nameValueVector.push_back( nameValue );
+ tmp.push_back( nameValue );
}
}
rtl_uString_release( pArg );
- }
- pNameValueVector = &nameValueVector;
- }
+ };
+ return tmp;
+ }();
bool found = false;
- for(NameValueVector::iterator ii = pNameValueVector->begin();
- ii != pNameValueVector->end();
+ for(NameValueVector::iterator ii = nameValueVector.begin();
+ ii != nameValueVector.end();
++ii)
{
if ((*ii).sName == key)
diff --git a/sal/rtl/hash.cxx b/sal/rtl/hash.cxx
index e12095f3d9e5..489030f62861 100644
--- a/sal/rtl/hash.cxx
+++ b/sal/rtl/hash.cxx
@@ -40,12 +40,7 @@ static void rtl_str_hash_free(StringHashTable *pHash);
static StringHashTable * getHashTable()
{
- static StringHashTable *pInternPool = nullptr;
- if (!pInternPool)
- {
- static StringHashTable* pHash = rtl_str_hash_new(1024);
- pInternPool = pHash;
- }
+ static StringHashTable* pInternPool = rtl_str_hash_new(1024);
return pInternPool;
}
More information about the Libreoffice-commits
mailing list