diff options
| author | Tor Lillqvist <tlillqvist@suse.com> | 2011-10-06 09:36:07 +0300 |
|---|---|---|
| committer | Tor Lillqvist <tlillqvist@suse.com> | 2011-10-06 09:36:07 +0300 |
| commit | cce8278ebaf333024b867d9b68d3feb70235f1dc (patch) | |
| tree | ff9c7b91b8230bd12a4782d8e71fe9c295281fc9 /basic | |
| parent | f4e93410603044f5796019e4a9f7845ec83d2b95 (diff) | |
WaE: unreachable code
Diffstat (limited to 'basic')
| -rw-r--r-- | basic/source/sbx/sbxcurr.cxx | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/basic/source/sbx/sbxcurr.cxx b/basic/source/sbx/sbxcurr.cxx index 574c8048088e..c227681795a4 100644 --- a/basic/source/sbx/sbxcurr.cxx +++ b/basic/source/sbx/sbxcurr.cxx @@ -268,13 +268,21 @@ start: case SbxSALINT64: { nRes = p->nInt64 * CURRENCY_FACTOR; break; +#if 0 + // Huh, is the 'break' above intentional? That means this + // is unreachable, obviously. Avoid warning by ifdeffing + // this out for now. Do not delete this #if 0 block unless + // you know for sure the 'break' above is intentional. if ( nRes > SbxMAXSALINT64 ) { SbxBase::SetError( SbxERR_OVERFLOW ); nRes = SbxMAXSALINT64; } +#endif } case SbxSALUINT64: nRes = p->nInt64 * CURRENCY_FACTOR; break; +#if 0 + // As above if ( nRes > SbxMAXSALINT64 ) { SbxBase::SetError( SbxERR_OVERFLOW ); nRes = SbxMAXSALINT64; @@ -284,6 +292,7 @@ start: SbxBase::SetError( SbxERR_OVERFLOW ); nRes = SbxMINSALINT64; } break; +#endif //TODO: bring back SbxINT64 types here for limits -1 with flag value at SAL_MAX/MIN case SbxSINGLE: if( p->nSingle * CURRENCY_FACTOR + 0.5 > (float)SAL_MAX_INT64 |