[Libreoffice-commits] core.git: Branch 'libreoffice-4-3-3' - vcl/source

Caolán McNamara caolanm at redhat.com
Wed Oct 15 01:23:07 PDT 2014


 vcl/source/outdev/text.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 33028a6362527537899f92eb5f1ca77ea2198544
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Thu Oct 9 11:13:10 2014 +0100

    Resolves: fdo#84809 crash when button is too short for text
    
    regression from 4c539fac018dfd44cd8db52161a8cb930c627da7
    
    commit 4c539fac018dfd44cd8db52161a8cb930c627da7
    Date:   Tue Dec 17 05:18:35 2013 -0600
        vcl get rid of xub_StrLen and STRING_LEN in outdev3
    
    before this there was a dubious cast which hid the problem
    
    Change-Id: I7c01c0f0ac211196a62acaf02297f29ae829403d
    (cherry picked from commit 3144b8deaf8397d719b7ad7a56b309655a53d90a)
    Reviewed-on: https://gerrit.libreoffice.org/11871
    Reviewed-by: David Tardon <dtardon at redhat.com>
    Tested-by: David Tardon <dtardon at redhat.com>
    (cherry picked from commit b3e96abc32623891a7147c9f8fa6477ffcddd643)
    Reviewed-on: https://gerrit.libreoffice.org/11943
    Reviewed-by: Miklos Vajna <vmiklos at collabora.co.uk>
    Reviewed-by: Andras Timar <andras.timar at collabora.com>
    Tested-by: Andras Timar <andras.timar at collabora.com>

diff --git a/vcl/source/outdev/text.cxx b/vcl/source/outdev/text.cxx
index b7bb028..e426368 100644
--- a/vcl/source/outdev/text.cxx
+++ b/vcl/source/outdev/text.cxx
@@ -1496,7 +1496,7 @@ void OutputDevice::ImplDrawText( OutputDevice& rTargetDevice, const Rectangle& r
             long nMaxTextWidth = ImplGetTextLines( aMultiLineInfo, nWidth, aStr, nStyle, _rLayout );
             nLines = (sal_Int32)(nHeight/nTextHeight);
             nFormatLines = aMultiLineInfo.Count();
-            if ( !nLines )
+            if (nLines <= 0)
                 nLines = 1;
             if ( nFormatLines > nLines )
             {


More information about the Libreoffice-commits mailing list