diff options
author | Caolán McNamara <caolanm@redhat.com> | 2014-11-17 12:40:45 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2014-11-17 14:53:24 +0000 |
commit | 6b709b27b3552ced17b612b78f1c8cadc81f620c (patch) | |
tree | c1538bf0c97abbd9969d2ea3dee994c43fd09a01 /basic/source | |
parent | OSL_ENSURE->assert where guaranteed deref follows (diff) | |
download | core-6b709b27b3552ced17b612b78f1c8cadc81f620c.tar.gz core-6b709b27b3552ced17b612b78f1c8cadc81f620c.zip |
coverity#735399 Logically dead code
we can only enter the bAssumeExprLParenMode controlled block once
because the block sets bAssumeExprLParenMode to false
bAssumeExprLParenMode is only true if the first token read in the
method is LPAREN
so we can only enter this block if the current token is LPAREN and
its the first token processed, so the token can't be BYVAL
Change-Id: I6e668fca4d127d7dbfe447e2d9f2231e05278a7d
Diffstat (limited to 'basic/source')
-rw-r--r-- | basic/source/comp/exprtree.cxx | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/basic/source/comp/exprtree.cxx b/basic/source/comp/exprtree.cxx index d004b0974010..c01a729125d9 100644 --- a/basic/source/comp/exprtree.cxx +++ b/basic/source/comp/exprtree.cxx @@ -1071,10 +1071,6 @@ SbiParameters::SbiParameters( SbiParser* p, bool bStandaloneExpression, bool bPa { bBracket = true; delete pExpr; - if( bByVal ) - { - pParser->Error( SbERR_LVALUE_EXPECTED ); - } return; } } |