core.git: i18npool/source

[Bogdan Buzea (via logerrit)]

 i18npool/source/breakiterator/breakiteratorImpl.cxx |   19 +------------------
 1 file changed, 1 insertion(+), 18 deletions(-)

New commits:
commit b1a6af0edd517e073545b89114169467c666a761
Author:     Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Mon Oct 21 08:05:19 2024 +0200
Commit:     David Gilbert <freedesk...@treblig.org>
CommitDate: Sun Nov 3 13:20:52 2024 +0100

    tdf#163486: PVS: Identical branches
    
    V1037 Two or more case-branches perform the same actions. Check lines: 82, 
118
    
    Change-Id: Iae91b2079c1c1d5aeb3000495364f3995266cbe4
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175274
    Reviewed-by: David Gilbert <freedesk...@treblig.org>
    Tested-by: Jenkins

diff --git a/i18npool/source/breakiterator/breakiteratorImpl.cxx 
b/i18npool/source/breakiterator/breakiteratorImpl.cxx
index 0f23a59cc1e9..6b6870f83ecd 100644
--- a/i18npool/source/breakiterator/breakiteratorImpl.cxx
+++ b/i18npool/source/breakiterator/breakiteratorImpl.cxx
@@ -79,6 +79,7 @@ static sal_Int32 skipSpace(std::u16string_view Text, 
sal_Int32 nPos, sal_Int32 l
     sal_Int32 pos=nPos;
     switch (rWordType) {
         case WordType::ANYWORD_IGNOREWHITESPACES:
+        case WordType::WORD_COUNT:
             if (bDirection)
                 while (nPos < len)
                 {
@@ -114,24 +115,6 @@ static sal_Int32 skipSpace(std::u16string_view Text, 
sal_Int32 nPos, sal_Int32 l
                     nPos = pos;
                 }
             break;
-        case WordType::WORD_COUNT:
-            if (bDirection)
-                while (nPos < len)
-                {
-                    ch = o3tl::iterateCodePoints(Text, &pos);
-                    if (!u_isUWhiteSpace(ch) && !isZWSP(ch))
-                        break;
-                    nPos = pos;
-                }
-            else
-                while (nPos > 0)
-                {
-                    ch = o3tl::iterateCodePoints(Text, &pos, -1);
-                    if (!u_isUWhiteSpace(ch) && !isZWSP(ch))
-                        break;
-                    nPos = pos;
-                }
-            break;
     }
     return nPos;
 }