diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2015-02-11 09:01:23 +0100 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2015-02-11 09:01:23 +0100 |
commit | 3ff852364b37f5ca7a7670e6a93907044b8769ae (patch) | |
tree | fdfdc1b2ad9ee3faba0f3fa70ef4aa1d4c6a9a34 /sal | |
parent | 1b046598e02bbf479a9a13ed61a5760944c5c891 (diff) |
coverity#1267660 etc.: remove redundant checks
...all these functions take a "handle to a created condition."
Change-Id: I043f310801452b05e6daaca4d246b305d18f6104
Diffstat (limited to 'sal')
-rw-r--r-- | sal/osl/unx/conditn.cxx | 20 |
1 files changed, 0 insertions, 20 deletions
diff --git a/sal/osl/unx/conditn.cxx b/sal/osl/unx/conditn.cxx index f5fedd4204a1..37040bff16cb 100644 --- a/sal/osl/unx/conditn.cxx +++ b/sal/osl/unx/conditn.cxx @@ -105,11 +105,6 @@ sal_Bool SAL_CALL osl_setCondition(oslCondition Condition) assert(Condition); pCond = (oslConditionImpl*)Condition; - if ( pCond == 0 ) - { - return sal_False; - } - nRet = pthread_mutex_lock(&pCond->m_Lock); if ( nRet != 0 ) { @@ -149,11 +144,6 @@ sal_Bool SAL_CALL osl_resetCondition(oslCondition Condition) pCond = (oslConditionImpl*)Condition; - if ( pCond == 0 ) - { - return sal_False; - } - nRet = pthread_mutex_lock(&pCond->m_Lock); if ( nRet != 0 ) { @@ -186,11 +176,6 @@ oslConditionResult SAL_CALL osl_waitCondition(oslCondition Condition, const Time SAL_INFO( "sal.osl.condition", "osl_waitCondition(" << pCond << ")" ); - if ( pCond == 0 ) - { - return osl_cond_result_error; - } - nRet = pthread_mutex_lock(&pCond->m_Lock); if ( nRet != 0 ) { @@ -270,11 +255,6 @@ sal_Bool SAL_CALL osl_checkCondition(oslCondition Condition) assert(Condition); pCond = (oslConditionImpl*)Condition; - if ( pCond == 0 ) - { - return sal_False; - } - nRet = pthread_mutex_lock(&pCond->m_Lock); SAL_WARN_IF( nRet != 0, "sal.osl.condition", "osl_checkCondition(" << pCond << "): pthread_mutex_lock failed: " << strerror(nRet) ); |