diff options
| author | bruvzg <7645683+bruvzg@users.noreply.github.com> | 2023-10-30 18:38:37 +0200 |
|---|---|---|
| committer | bruvzg <7645683+bruvzg@users.noreply.github.com> | 2023-10-31 08:41:35 +0200 |
| commit | f52559bf98cffe483919f3a4fe397c1c78a90f06 (patch) | |
| tree | c71b22b123698ca9df54373e8e56b68717340c74 /modules/text_server_adv | |
| parent | 93cdacbb0a30f12b2f3f5e8e06b90149deeb554b (diff) | |
| download | redot-engine-f52559bf98cffe483919f3a4fe397c1c78a90f06.tar.gz | |
[TextServer] Fix glyph comparator ambiguous output, causing "bad comparison function" error spam when processing text with excessive use of combining diacritics.
Diffstat (limited to 'modules/text_server_adv')
| -rw-r--r-- | modules/text_server_adv/text_server_adv.h | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/modules/text_server_adv/text_server_adv.h b/modules/text_server_adv/text_server_adv.h index 6ce2692fd3..cbd2911aaf 100644 --- a/modules/text_server_adv/text_server_adv.h +++ b/modules/text_server_adv/text_server_adv.h @@ -681,11 +681,7 @@ class TextServerAdvanced : public TextServerExtension { _FORCE_INLINE_ bool operator()(const Glyph &l, const Glyph &r) const { if (l.start == r.start) { if (l.count == r.count) { - if ((l.flags & TextServer::GRAPHEME_IS_VIRTUAL) == TextServer::GRAPHEME_IS_VIRTUAL) { - return false; - } else { - return true; - } + return (l.flags & TextServer::GRAPHEME_IS_VIRTUAL) < (r.flags & TextServer::GRAPHEME_IS_VIRTUAL); } return l.count > r.count; // Sort first glyph with count & flags, order of the rest are irrelevant. } else { |