diff options
| author | Luboš Luňák <l.lunak@collabora.com> | 2022-08-11 15:02:02 +0200 |
|---|---|---|
| committer | Luboš Luňák <l.lunak@collabora.com> | 2022-08-12 12:13:32 +0200 |
| commit | 82553b46b689c0ff4c218a2a70918c3f4dafccfe (patch) | |
| tree | e72f5ddb9a3c630aee368d3092ab0f20dcc1937b /vcl/source/gdi/impglyphitem.cxx | |
| parent | d21a23cb03b54d423faf6c4a3d706cb9c749916d (diff) | |
fix checking glyph break position (tdf#150138)
Glyphs in RTL mode are in reverse, which includes the fast that
a cluster start is last, not first. So going back requires means
going back just one glyph, not a whole cluster. I'm not sure why
I originally did it this way, tdf#148954 works fine even this way,
so presumably I got confused.
Change-Id: I6ed0caf4e64e93e0643f7b88099bc7c992ea3329
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138144
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
Diffstat (limited to 'vcl/source/gdi/impglyphitem.cxx')
| -rw-r--r-- | vcl/source/gdi/impglyphitem.cxx | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/vcl/source/gdi/impglyphitem.cxx b/vcl/source/gdi/impglyphitem.cxx index 94da485276ae..51bff5966d32 100644 --- a/vcl/source/gdi/impglyphitem.cxx +++ b/vcl/source/gdi/impglyphitem.cxx @@ -191,10 +191,6 @@ bool SalLayoutGlyphsImpl::isSafeToBreak(const_iterator pos, bool rtl) const if (pos == begin()) return true; --pos; - while (pos >= begin() && (pos->IsInCluster() && !pos->IsClusterStart())) - --pos; - if (pos < begin()) - return true; } // Don't create a subset if it's not safe to break at the beginning or end of the sequence // (https://harfbuzz.github.io/harfbuzz-hb-buffer.html#hb-glyph-flags-t). |