walkero
/
harfbuzz
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

[ot-layout] Don't remove_output() before reverse substitution 

No need anymore, because of new swap_buffers() semantics.

Just assert instead.
This commit is contained in:
Behdad Esfahbod 2021-07-12 17:02:03 -06:00
parent 10a9960f0a
commit 3807061d63
3 changed files with 1 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/hb-buffer.cc
View File

@ -281,16 +281,6 @@ hb_buffer_t::add_info (const hb_glyph_info_t &glyph_info)
} }
void
hb_buffer_t::remove_output ()
{
have_output = false;
have_positions = false;
out_len = 0;
out_info = info;
}
void void
hb_buffer_t::clear_output () hb_buffer_t::clear_output ()
{ {

1
src/hb-buffer.hh
View File

@ -209,7 +209,6 @@ struct hb_buffer_t
HB_INTERNAL void guess_segment_properties (); HB_INTERNAL void guess_segment_properties ();
HB_INTERNAL void swap_buffers (); HB_INTERNAL void swap_buffers ();
HB_INTERNAL void remove_output ();
HB_INTERNAL void clear_output (); HB_INTERNAL void clear_output ();
HB_INTERNAL void clear_positions (); HB_INTERNAL void clear_positions ();

4
src/hb-ot-layout.cc
View File

@ -1900,10 +1900,8 @@ apply_string (OT::hb_ot_apply_context_t *c,
else else
{ {
/* in-place backward substitution/positioning */ /* in-place backward substitution/positioning */
if (Proxy::table_index == 0u) assert (!buffer->have_output);
buffer->remove_output ();
buffer->idx = buffer->len - 1; buffer->idx = buffer->len - 1;
apply_backward (c, accel); apply_backward (c, accel);
} }
} }