sc/source/filter/inc/lotform.hxx | 2 +-
sc/source/filter/lotus/lotform.cxx | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
New commits:
commit dfaee8d8f43f8725a0e999279f99e552eeb0bee2
Author: Xisco Fauli <xiscofa...@libreoffice.org>
AuthorDate: Wed Oct 23 13:56:41 2024 +0200
Commit: Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Wed Oct 23 15:32:09 2024 +0200
tdf#163486: PVS: enum type as bool
V768 The variable 'eType' is of enum type. It is odd that it is used as
a variable of a Boolean-type.
Change-Id: I38f0302d8edcaef144f6071dc86500937d305591
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175478
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>
diff --git a/sc/source/filter/inc/lotform.hxx b/sc/source/filter/inc/lotform.hxx
index 29751a2f5533..fa618b021da0 100644
--- a/sc/source/filter/inc/lotform.hxx
+++ b/sc/source/filter/inc/lotform.hxx
@@ -23,7 +23,7 @@
enum FUNC_TYPE
{
- FT_Return = 0, // End Formula
+ FT_Return, // End Formula
FT_FuncFix0, // Function, 0 Parameter
FT_FuncFix1, // Function, 0 Parameter
FT_FuncFix2, // Function, 0 Parameter
diff --git a/sc/source/filter/lotus/lotform.cxx
b/sc/source/filter/lotus/lotform.cxx
index b5f9ee055a7a..10bfc67e2da3 100644
--- a/sc/source/filter/lotus/lotform.cxx
+++ b/sc/source/filter/lotus/lotform.cxx
@@ -426,7 +426,7 @@ void LotusToSc::Convert( std::unique_ptr<ScTokenArray>&
rpErg, sal_Int32& rRest
nBytesLeft = rRest;
- while( eType ) // != FT_Return (==0)
+ while( eType != FT_Return )
{
sal_uInt8 nOc;
Read(nOc);
