summaryrefslogtreecommitdiff
path: root/formula
diff options
context:
space:
mode:
authorCaolán McNamara <caolanm@redhat.com>2011-04-12 20:21:44 +0100
committerCaolán McNamara <caolanm@redhat.com>2011-04-12 21:23:29 +0100
commitd283eadbc5b82ced160017b0870192305fb70e71 (patch)
tree426f64f0e53ba53b5db02d37deded83fa9b34625 /formula
parenta1885cb7327c0f2b3b1c46a6508adccfc4115196 (diff)
WaE: various compiler warnings
Diffstat (limited to 'formula')
-rw-r--r--formula/source/core/api/FormulaCompiler.cxx9
-rw-r--r--formula/source/core/api/token.cxx9
2 files changed, 14 insertions, 4 deletions
diff --git a/formula/source/core/api/FormulaCompiler.cxx b/formula/source/core/api/FormulaCompiler.cxx
index 0da88cd678d0..bd38fca34798 100644
--- a/formula/source/core/api/FormulaCompiler.cxx
+++ b/formula/source/core/api/FormulaCompiler.cxx
@@ -722,6 +722,7 @@ OpCode FormulaCompiler::GetEnglishOpCode( const String& rName ) const
bool FormulaCompiler::IsOpCodeVolatile( OpCode eOp )
{
+ bool bRet = false;
switch (eOp)
{
// no parameters:
@@ -739,9 +740,13 @@ bool FormulaCompiler::IsOpCodeVolatile( OpCode eOp )
case ocIndirectXL:
// ocOffset results in indirect references.
case ocOffset:
- return true;
+ bRet = true;
+ break;
+ default:
+ bRet = false;
+ break;
}
- return false;
+ return bRet;
}
// Remove quotes, escaped quotes are unescaped.
diff --git a/formula/source/core/api/token.cxx b/formula/source/core/api/token.cxx
index cec4f58ee9dd..8b8148d035ac 100644
--- a/formula/source/core/api/token.cxx
+++ b/formula/source/core/api/token.cxx
@@ -135,14 +135,19 @@ bool FormulaToken::IsMatrixFunction() const
bool FormulaToken::IsExternalRef() const
{
+ bool bRet = false;
switch (eType)
{
case svExternalSingleRef:
case svExternalDoubleRef:
case svExternalName:
- return true;
+ bRet = true;
+ break;
+ default:
+ bRet = false;
+ break;
}
- return false;
+ return bRet;
}
bool FormulaToken::operator==( const FormulaToken& rToken ) const