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

[Jan-Marek Glogowski (via logerrit)]

 vcl/qt5/Qt5Graphics_Text.cxx |    4 ++++
 1 file changed, 4 insertions(+)

New commits:
commit feb5985b463974476985dee83376067941d77da0
Author:     Jan-Marek Glogowski <glogow at fbihome.de>
AuthorDate: Tue Oct 27 16:41:30 2020 +0100
Commit:     Jan-Marek Glogowski <glogow at fbihome.de>
CommitDate: Tue Oct 27 20:19:23 2020 +0100

    tdf#137804 Qt5 don't paint empty strings
    
    For a test I added a SAL_WARN, but that created many more debug
    messages then expected, just when opening the print dialog. So
    this just handles the crash like the CairoTextRender variant of
    DrawTextLayout does, by returning early.
    
    Change-Id: Ie5b58ebbba02dace0e777b60fcbc3b9f46ea9d66
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104893
    Tested-by: Jenkins
    Reviewed-by: Jan-Marek Glogowski <glogow at fbihome.de>

diff --git a/vcl/qt5/Qt5Graphics_Text.cxx b/vcl/qt5/Qt5Graphics_Text.cxx
index 431ab4e32f3d..abf71b7ec7e9 100644
--- a/vcl/qt5/Qt5Graphics_Text.cxx
+++ b/vcl/qt5/Qt5Graphics_Text.cxx
@@ -317,6 +317,10 @@ void Qt5Graphics::DrawTextLayout(const GenericSalLayout& rLayout)
         positions.push_back(QPointF(aPos.X(), aPos.Y()));
     }
 
+    // seems to be common to try to layout an empty string...
+    if (positions.empty())
+        return;
+
     if (nOrientation)
         pQt5Layout->SetOrientation(nOrientation);