Bug 118577 - Corruption in letters of Autocorrect Options -> Replace tab pair listings
Summary: Corruption in letters of Autocorrect Options -> Replace tab pair listings
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
6.1.0.1 rc
Hardware: All Windows (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, regression
Depends on:
Blocks: CommonSalLayout-refactoring-regressions
  Show dependency treegraph
 
Reported: 2018-07-06 12:43 UTC by Marco A.G.Pinto
Modified: 2018-07-10 14:03 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
corrupted characters screenshot (25.16 KB, image/png)
2018-07-06 12:43 UTC, Marco A.G.Pinto
Details
About window (18.55 KB, image/png)
2018-07-06 13:08 UTC, Marco A.G.Pinto
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Marco A.G.Pinto 2018-07-06 12:43:53 UTC
Created attachment 143348 [details]
corrupted characters screenshot

Hello!

The symbols/letters have lots of corrupted characters.

Please see the attachment.
Comment 1 Marco A.G.Pinto 2018-07-06 13:08:27 UTC
Created attachment 143350 [details]
About window
Comment 2 V Stuart Foote 2018-07-06 14:12:01 UTC
Confirmed on Windows 10 64-bit both with recent master and 6.1.0 rc1

Not noted with OpenGL rendering, but prevelent with default GDI+ rendering.

STR.

1. New writer document
2. open Tools -> AutoCorrect -> AutoCorrect options dialog
3. select the "Replace" tab
4. scroll slowly down the list of replacement pairs
5. starts correct, but soon start having corrupt "replace" entry strings in the emoji : bracketed value. Few if any dropped glyphs in the "with" entries.

reaching the plain text corrections, errors go away

And if restarting with OpenGL enabled, do not see the corruption of the "replace" strings.
Comment 3 Xisco Faulí 2018-07-06 14:35:11 UTC
Regression introduced by:

author	Jan-Marek Glogowski <glogow@fbihome.de>	2017-12-26 15:14:31 +0000
committer	Khaled Hosny <khaledhosny@eglug.org>	2018-05-08 00:55:27 +0200
commit bdccb7e9991d83029eb2f2f11327b54534a00db8 (patch)
tree c32e95c49849647dc72c1071f375f3d2b67d8d7a
parent 9615e45d2e2bac79c252a018846e4f20012cfa34 (diff)
Refactor CommonSalLayout font handling
Moves all platform specific code from CommonSalLayout into the
platform specific plugins. This way the vcl library won't depend
on the Qt5 libraries and the Qt5Font header can be moved into the
qt5 VCL plugin.

While at it, switch the CommonSalLayouts font reference from the
FontSelectPattern to the LogicalFontInstance and also add the
harfbuzz font handling to the instance.

Bisected with: bibisect-win32-6.1

Adding Cc: to Jan-Marek Glogowski
Comment 4 Roman Kuznetsov 2018-07-06 14:56:27 UTC
Confirm for Russian GUI and locale in 6.1 beta 2
Comment 5 Xisco Faulí 2018-07-10 13:58:49 UTC
Verified in

Version: 6.2.0.0.alpha0+
Build ID: c290f692dd28094d41dff686f3faa1c4e14b556e
CPU threads: 16; OS: Windows 6.3; UI render: default; 
Locale: en-GB (en_GB); Calc: group threaded

@Jan-Marek, thanks for fixing this! Should it be closed as RESOLVED FIXED?