From 895843ad466954b63f6f99a1b6319b7813d0dbe1 Mon Sep 17 00:00:00 2001 From: Khaled Hosny Date: Sat, 13 Aug 2022 00:00:27 +0200 Subject: tdf#103492: Messed Arabic letter spacing in text starting with LTR character(s) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This partially reverts the still relevant parts of commit commit 41007842ed9bb5d6165792a197769f72dae55a2c Author: Martin Hosken Date: Thu Sep 10 10:14:18 2015 +0700 Refactor graphite integration and update graphite Change-Id: I09db7262c2a6180a078620ba086dd7a4c132bd82 Reviewed-on: https://gerrit.libreoffice.org/18459 Tested-by: Jenkins Reviewed-by: László Németh Reviewed-by: Martin Hosken No mention of the commit about this change or its relevance to Graphite integration. But the separate Graphite layout engine is long gone and Graphite handling goes through HarfBuzz now, so whatever the change was for is probably no longer relevant. I reverted only one the two changes in impedit3.cxx, as the other one does not seem to make any difference for this issue. Change-Id: Id70a711d6231cce52c9a32dbc3604860b1da9f4a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138217 Reviewed-by: Caolán McNamara Tested-by: Caolán McNamara --- editeng/source/editeng/impedit3.cxx | 2 -- 1 file changed, 2 deletions(-) (limited to 'editeng') diff --git a/editeng/source/editeng/impedit3.cxx b/editeng/source/editeng/impedit3.cxx index 6195b0e8e606..5ff196f02040 100644 --- a/editeng/source/editeng/impedit3.cxx +++ b/editeng/source/editeng/impedit3.cxx @@ -838,8 +838,6 @@ bool ImpEditEngine::CreateLines( sal_Int32 nPara, sal_uInt32 nStartPosY ) EditLine aSaveLine( *pLine ); SvxFont aTmpFont( pNode->GetCharAttribs().GetDefFont() ); - ImplInitLayoutMode(*GetRefDevice(), nPara, nIndex); - std::vector aBuf( pNode->Len() ); bool bSameLineAgain = false; // For TextRanger, if the height changes. -- cgit