[Libreoffice-commits] core.git: vcl/quartz

[Stephan Bergmann (via logerrit)]

 vcl/quartz/ctfonts.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit d0f3e24a7a34aac41e0b433d45c657b136ff8fe0
Author:     Stephan Bergmann <sberg...@redhat.com>
AuthorDate: Tue Oct 11 08:09:51 2022 +0200
Commit:     Stephan Bergmann <sberg...@redhat.com>
CommitDate: Tue Oct 11 09:12:17 2022 +0200

    loplugin:loopvartoosmall
    
    > /Users/stephan/lo/core/vcl/quartz/ctfonts.cxx:255:33: error: loop index 
type 'unsigned int' is narrower than length type 'CFIndex' (aka 'long') 
[loplugin:loopvartoosmall]
    >             for (auto i = 0u; i < nTags; i++)
    >                               ~~^~~~~~~
    
    (and whatever the motivation in d7083fe6dd383ac4144fbe53e300bc3d34f26ef6
    "tdf#72456: Support font embedding on macOS" to use an unsigned 0u here)
    
    Change-Id: I6a96dd2b2146c27d68a8a53daf3c2fe1f744738d
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/141201
    Reviewed-by: خالد حسني <kha...@aliftype.com>
    Tested-by: Jenkins

diff --git a/vcl/quartz/ctfonts.cxx b/vcl/quartz/ctfonts.cxx
index eb98b24569df..887abe4f2f35 100644
--- a/vcl/quartz/ctfonts.cxx
+++ b/vcl/quartz/ctfonts.cxx
@@ -252,7 +252,7 @@ hb_blob_t* CoreTextFontFace::GetHbTable(hb_tag_t nTag) const
                 return nullptr;
 
             hb_face_t* pHbFace = hb_face_builder_create();
-            for (auto i = 0u; i < nTags; i++)
+            for (CFIndex i = 0; i < nTags; i++)
             {
                 auto nTable = 
reinterpret_cast<intptr_t>(CFArrayGetValueAtIndex(pTags, i));
                 assert(nTable);