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

Caolán McNamara caolanm at redhat.com
Mon Oct 13 01:21:26 PDT 2014


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

New commits:
commit b3e96abc32623891a7147c9f8fa6477ffcddd643
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>

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