diff options
| author | Xisco Fauli <xiscofauli@libreoffice.org> | 2024-11-27 13:01:02 +0100 |
|---|---|---|
| committer | Xisco Fauli <xiscofauli@libreoffice.org> | 2024-11-27 14:23:29 +0100 |
| commit | ec68e9884a362693d5f5b1bbbe167bc72b585999 (patch) | |
| tree | b0c2c861dffe761ce4a200bf3a48a78ee71c4bbc | |
| parent | b54e905ce6d295104af9a6d2a01b38c02abee6e3 (diff) | |
PVS: V560 A part of conditional expression is always true: bEqual
Since
commit d0484bcdcbffd42d242647a2fdc1a203d176b775
Author: Jens-Heiner Rechtien <hr@openoffice.org>
Date: Mon Sep 18 23:16:46 2000 +0000
initial import
Change-Id: Ib1cf69ecf70bf9d866308892562a2d9080e1350f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177413
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
| -rw-r--r-- | sc/source/core/data/subtotalparam.cxx | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/sc/source/core/data/subtotalparam.cxx b/sc/source/core/data/subtotalparam.cxx index 6fd8e18c6b0e..ae28606d3643 100644 --- a/sc/source/core/data/subtotalparam.cxx +++ b/sc/source/core/data/subtotalparam.cxx @@ -155,9 +155,8 @@ bool ScSubTotalParam::operator==( const ScSubTotalParam& rOther ) const { for (SCCOL j=0; (j<nSubTotals[i]) && bEqual; j++) { - bEqual = bEqual - && (pSubTotals[i][j] == rOther.pSubTotals[i][j]) - && (pFunctions[i][j] == rOther.pFunctions[i][j]); + bEqual = pSubTotals[i][j] == rOther.pSubTotals[i][j] + && pFunctions[i][j] == rOther.pFunctions[i][j]; } } } |