sc/source/core/data/subtotalparam.cxx | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
New commits: commit ec68e9884a362693d5f5b1bbbe167bc72b585999 Author: Xisco Fauli <[email protected]> AuthorDate: Wed Nov 27 13:01:02 2024 +0100 Commit: Xisco Fauli <[email protected]> CommitDate: Wed Nov 27 14:23:29 2024 +0100 PVS: V560 A part of conditional expression is always true: bEqual Since commit d0484bcdcbffd42d242647a2fdc1a203d176b775 Author: Jens-Heiner Rechtien <[email protected]> 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 <[email protected]> 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]; } } }
