diff options
author | Caolán McNamara <caolanm@redhat.com> | 2011-04-12 20:21:44 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2011-04-12 21:23:29 +0100 |
commit | d283eadbc5b82ced160017b0870192305fb70e71 (patch) | |
tree | 426f64f0e53ba53b5db02d37deded83fa9b34625 /formula | |
parent | a1885cb7327c0f2b3b1c46a6508adccfc4115196 (diff) |
WaE: various compiler warnings
Diffstat (limited to 'formula')
-rw-r--r-- | formula/source/core/api/FormulaCompiler.cxx | 9 | ||||
-rw-r--r-- | formula/source/core/api/token.cxx | 9 |
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 |