diff options
| author | Bogdan Buzea <buzea.bogdan@libreoffice.org> | 2024-10-18 06:26:50 +0200 |
|---|---|---|
| committer | David Gilbert <freedesktop@treblig.org> | 2024-11-06 22:26:56 +0100 |
| commit | d98de2d0324c0e1cd808643633889423fccfc75e (patch) | |
| tree | b8a1bd9a8d25181c65c36f3e7ed673d933a0c6f5 /connectivity/source | |
| parent | 49f7a5209f890c497d15782ef4c3cc7c199bc0c8 (diff) | |
tdf#163486: PVS: Identical branches
Change-Id: Iad298c2819b0fa1fd4d98a64f19008a7ef8d39be
V1037: Two or more case-branches perform the same actions. Check lines: 353, 362
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175050
Reviewed-by: David Gilbert <freedesktop@treblig.org>
Tested-by: Jenkins
Diffstat (limited to 'connectivity/source')
| -rw-r--r-- | connectivity/source/drivers/file/fcomp.cxx | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/connectivity/source/drivers/file/fcomp.cxx b/connectivity/source/drivers/file/fcomp.cxx index 871e8fc8bef6..86ed7d4cf8c5 100644 --- a/connectivity/source/drivers/file/fcomp.cxx +++ b/connectivity/source/drivers/file/fcomp.cxx @@ -350,6 +350,8 @@ void OPredicateCompiler::execute_BETWEEN(OSQLParseNode const * pPredicateNode) break; case DataType::DECIMAL: case DataType::NUMERIC: + case DataType::DOUBLE: + case DataType::REAL: pOb1->setValue(pOb1->getValue().getDouble()); pOb2->setValue(pOb2->getValue().getDouble()); break; @@ -357,11 +359,6 @@ void OPredicateCompiler::execute_BETWEEN(OSQLParseNode const * pPredicateNode) pOb1->setValue(pOb1->getValue().getFloat()); pOb2->setValue(pOb2->getValue().getFloat()); break; - case DataType::DOUBLE: - case DataType::REAL: - pOb1->setValue(pOb1->getValue().getDouble()); - pOb2->setValue(pOb2->getValue().getDouble()); - break; case DataType::DATE: pOb1->setValue(pOb1->getValue().getDate()); pOb2->setValue(pOb2->getValue().getDate()); |