[Libreoffice-commits] core.git: 2 commits - helpcontent2 sal/rtl

[Mike Kaganski (via logerrit)]

 helpcontent2        |    2 +-
 sal/rtl/ustring.cxx |    4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 15cce9806cff49d54a80b047dec33b021080d5cb
Author:     Mike Kaganski <mike.kagan...@collabora.com>
AuthorDate: Tue Dec 13 09:25:47 2022 +0000
Commit:     Mike Kaganski <mike.kagan...@collabora.com>
CommitDate: Tue Dec 13 15:11:34 2022 +0000

    Fix the "Found char > 127" warning to check the correct characters
    
    Change-Id: Ia4a713c8a5ddda11e9802cbc317dde9ff48b8fd3
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144026
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>

diff --git a/sal/rtl/ustring.cxx b/sal/rtl/ustring.cxx
index 6fc0e87076c2..45ab6e166871 100644
--- a/sal/rtl/ustring.cxx
+++ b/sal/rtl/ustring.cxx
@@ -194,10 +194,10 @@ sal_Bool SAL_CALL 
rtl_ustr_asciil_reverseEquals_WithLength( const sal_Unicode* p
     const char*     pStr2Run = pStr2+nStrLen;
     while ( pStr1 < pStr1Run )
     {
-        SAL_WARN_IF( !rtl::isAscii(static_cast<unsigned char>(*pStr2)), 
"rtl.string",
-                    "rtl_ustr_asciil_reverseEquals_WithLength - Found char > 
127" );
         pStr1Run--;
         pStr2Run--;
+        SAL_WARN_IF( !rtl::isAscii(static_cast<unsigned char>(*pStr2Run)), 
"rtl.string",
+                    "rtl_ustr_asciil_reverseEquals_WithLength - Found char > 
127" );
         if( *pStr1Run != static_cast<sal_Unicode>(*pStr2Run) )
             return false;
     }
commit dc3b41f1d08479b9732b065b88f8a02512a96e47
Author:     Christian Lohmaier <cl...@documentfoundation.org>
AuthorDate: Tue Dec 13 16:11:32 2022 +0100
Commit:     Gerrit Code Review <ger...@gerrit.libreoffice.org>
CommitDate: Tue Dec 13 15:11:32 2022 +0000

    Update git submodules
    
    * Update helpcontent2 from branch 'master'
      to 5b99afe8ba3afe1a4673f8b7628011ced5583570
      - avoid needless rebuilds of helpimages
    
        also the create_ilist.pl doesn't really do much anymore, was easily
        replaced by a plain $(file …) call
    
        Change-Id: I63a41ef8ed289feb4ab210961d2200032adaac28
        Reviewed-on: https://gerrit.libreoffice.org/c/help/+/143936
        Tested-by: Jenkins
        Reviewed-by: Christian Lohmaier <lohmaier+libreoff...@googlemail.com>

diff --git a/helpcontent2 b/helpcontent2
index e1cf51797657..5b99afe8ba3a 160000
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit e1cf5179765728ac70b100352ffb09bc1ec3fde5
+Subproject commit 5b99afe8ba3afe1a4673f8b7628011ced5583570