Commited and pushed on master.
Now it's :
bool WinSalInstance::CheckYieldMutex()
{
bool bRet = true;
SalData* pSalData = GetSalData();
DWORD nCurThreadId = GetCurrentThreadId();
if ( pSalData->mpFirstInstance )
{
SalYieldMutex* pYieldMutex =
pSalData->mpFirstInstance->mpSalYieldMutex;
bRet = (pYieldMutex->mnThreadId == nCurThreadId);
}
return bRet;
}
I wonder if we could more simplify like this :
bool WinSalInstance::CheckYieldMutex()
{
bool bRet = true;
SalData* pSalData = GetSalData();
if ( pSalData->mpFirstInstance )
{
SalYieldMutex* pYieldMutex =
pSalData->mpFirstInstance->mpSalYieldMutex;
bRet = (pYieldMutex->mnThreadId == (GetCurrentThreadId()))
}
return bRet;
}
Anyway, thank you Michael !
--
View this message in context:
http://nabble.documentfoundation.org/Duplicate-expression-in-vcl-win-source-app-salinst-cxx-tp3695749p3700758.html
Sent from the Dev mailing list archive at Nabble.com.
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
