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

[Khaled Hosny]

 vcl/quartz/ctfonts.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 04d086531ff4a3ab24b704489270c7c23792bd20
Author: Khaled Hosny <khaledhosny at eglug.org>
Date:   Wed Feb 22 23:56:19 2017 +0200

    tdf#104509: Fix GetGlyphBoundRect on Mac
    
    The rectangle top and bottom were mixed up! I tried to fix this in
    8cef9b4dbafdebc0566d6d7f715f27f978ddfe5b, but it was not the right fix.
    
    Change-Id: Ia033ff2823bfd641e830d748d413d7791bfee5b8
    Reviewed-on: https://gerrit.libreoffice.org/34557
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Khaled Hosny <khaledhosny at eglug.org>

diff --git a/vcl/quartz/ctfonts.cxx b/vcl/quartz/ctfonts.cxx
index 1c60db3..b5db7e5 100644
--- a/vcl/quartz/ctfonts.cxx
+++ b/vcl/quartz/ctfonts.cxx
@@ -167,9 +167,9 @@ bool CoreTextStyle::GetGlyphBoundRect(const GlyphItem& rGlyph, Rectangle& rRect
         aCGRect = CGRectApplyAffineTransform(aCGRect, CGAffineTransformMakeRotation(mfFontRotation));
 
     rRect.Left()   = lrint( aCGRect.origin.x );
-    rRect.Top()    = lrint(-aCGRect.origin.y );
+    rRect.Top()    = lrint(-aCGRect.origin.y - aCGRect.size.height );
     rRect.Right()  = lrint( aCGRect.origin.x + aCGRect.size.width );
-    rRect.Bottom() = lrint( aCGRect.origin.y - aCGRect.size.height );
+    rRect.Bottom() = lrint(-aCGRect.origin.y );
     return true;
 }