diff options
| author | Eli Zaretskii <eliz@gnu.org> | 2024-02-27 13:12:15 +0200 |
|---|---|---|
| committer | Eli Zaretskii <eliz@gnu.org> | 2024-02-27 13:12:15 +0200 |
| commit | 383ccf6d51fd7af65dbcc1ce159a03369a48d27f (patch) | |
| tree | c3cefc3185a74f7518a57f9e0fee19c8330b7757 | |
| parent | b7cef701cb587ecb66f192e4d41aa202645560e0 (diff) | |
| download | emacs-383ccf6d51fd7af65dbcc1ce159a03369a48d27f.tar.gz | |
Avoid assertion violations in bidi.c
* src/bidi.c (bidi_resolve_brackets): Move assertion about
'resolved_level' to where it belongs. This avoids unnecessary
aborts when the character is not a bracket type and doesn't need
BPA resolution. (Bug#69421)
| -rw-r--r-- | src/bidi.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/bidi.c b/src/bidi.c index 93bb061ac32..90c0061549a 100644 --- a/src/bidi.c +++ b/src/bidi.c @@ -2908,7 +2908,6 @@ bidi_resolve_brackets (struct bidi_it *bidi_it) } else if (bidi_it->bracket_pairing_pos != eob) { - eassert (bidi_it->resolved_level == -1); /* If the cached state shows an increase of embedding level due to an isolate initiator, we need to update the 1st cached state of the next run of the current isolating sequence with @@ -2917,6 +2916,7 @@ bidi_resolve_brackets (struct bidi_it *bidi_it) if (bidi_it->level_stack[bidi_it->stack_idx].level > prev_level && ISOLATE_STATUS (bidi_it, bidi_it->stack_idx)) { + eassert (bidi_it->resolved_level == -1); bidi_record_type_for_neutral (&prev_for_neutral, prev_level, 0); bidi_record_type_for_neutral (&next_for_neutral, prev_level, 1); } @@ -2931,6 +2931,7 @@ bidi_resolve_brackets (struct bidi_it *bidi_it) } else if (bidi_it->bracket_pairing_pos == -1) { + eassert (bidi_it->resolved_level == -1); /* Higher levels were not BPA-resolved yet, even if cached by bidi_find_bracket_pairs. Force application of BPA to the new level now. */ |