Accepting request 900650 from LibreOffice:7.1
- Fix tdf#142839: a regression caused by "Multi column textbox in editengine" * tdf142839.patch OBS-URL: https://build.opensuse.org/request/show/900650 OBS-URL: https://build.opensuse.org/package/show/LibreOffice:Factory/libreoffice?expand=0&rev=971
This commit is contained in:
parent
56a290ddc4
commit
b173291d2e
@ -1,3 +1,9 @@
|
||||
-------------------------------------------------------------------
|
||||
Thu Jun 17 19:38:40 UTC 2021 - Andras Timar <andras.timar@collabora.com>
|
||||
|
||||
- Fix tdf#142839: a regression caused by "Multi column textbox in editengine"
|
||||
* tdf142839.patch
|
||||
|
||||
-------------------------------------------------------------------
|
||||
Tue Jun 15 12:28:58 UTC 2021 - Guillaume GARDET <guillaume.gardet@opensuse.org>
|
||||
|
||||
|
@ -109,6 +109,8 @@ Patch7: bsc1185505.patch
|
||||
Patch8: bsc1185797.patch
|
||||
# bsc#1182969 Multi column textbox in editengine
|
||||
Patch9: bsc1182969.patch
|
||||
# tdf#142839 Fix a regression caused by "Multi column textbox in editengine"
|
||||
Patch10: tdf142839.patch
|
||||
# Build with java 8
|
||||
Patch101: 0001-Revert-java-9-changes.patch
|
||||
# try to save space by using hardlinks
|
||||
@ -984,6 +986,7 @@ Provides %{langname} translations and additional resources (help files, etc.) fo
|
||||
%patch7 -p1
|
||||
%patch8 -p1
|
||||
%patch9 -p1
|
||||
%patch10 -p1
|
||||
%if 0%{?suse_version} < 1500
|
||||
%patch101 -p1
|
||||
%endif
|
||||
|
38
tdf142839.patch
Normal file
38
tdf142839.patch
Normal file
@ -0,0 +1,38 @@
|
||||
From 4a2894a2b9c973b275788190220d205e41eb92db Mon Sep 17 00:00:00 2001
|
||||
From: Mike Kaganski <mike.kaganski@collabora.com>
|
||||
Date: Thu, 17 Jun 2021 17:22:36 +0300
|
||||
Subject: [PATCH] tdf#142839: do not forget to initialize *pHeightNTP
|
||||
|
||||
Regression after commit d0a1616ccad0dd5f5a02c1b0204f537b57d0b4b5
|
||||
|
||||
Change-Id: Iaa8f67e0c7e3d99b057294717d24ffac4ad3534c
|
||||
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117389
|
||||
Tested-by: Jenkins
|
||||
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
||||
(cherry picked from commit 59611f7b6acbf643bde0e0121ee51e2df53e7d50)
|
||||
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117305
|
||||
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
|
||||
---
|
||||
|
||||
diff --git a/editeng/source/editeng/impedit2.cxx b/editeng/source/editeng/impedit2.cxx
|
||||
index 4980c57..ff080b1 100644
|
||||
--- a/editeng/source/editeng/impedit2.cxx
|
||||
+++ b/editeng/source/editeng/impedit2.cxx
|
||||
@@ -3437,6 +3437,8 @@
|
||||
tools::Long ImpEditEngine::Calc1ColumnTextHeight(tools::Long* pHeightNTP)
|
||||
{
|
||||
tools::Long nHeight = 0;
|
||||
+ if (pHeightNTP)
|
||||
+ *pHeightNTP = 0;
|
||||
// Pretend that we have ~infinite height to get total height
|
||||
comphelper::ValueRestorationGuard aGuard(nCurTextHeight,
|
||||
std::numeric_limits<tools::Long>::max());
|
||||
@@ -3467,6 +3469,8 @@
|
||||
tools::Long nTentativeColHeight = mnMinColumnWrapHeight;
|
||||
tools::Long nWantedIncrease = 0;
|
||||
tools::Long nCurrentTextHeight;
|
||||
+ if (pHeightNTP)
|
||||
+ *pHeightNTP = 0;
|
||||
|
||||
// This does the necessary column balancing for the case when the text does not fit min height.
|
||||
// When the height of column (taken from nCurTextHeight) is too small, the last column will
|
Loading…
x
Reference in New Issue
Block a user