diff --git a/src/harfbuzz-buffer.c b/src/harfbuzz-buffer.c index f597ac9af..e39a24b1d 100644 --- a/src/harfbuzz-buffer.c +++ b/src/harfbuzz-buffer.c @@ -51,10 +51,15 @@ hb_buffer_ensure( HB_Buffer buffer, while (size > new_allocated) new_allocated += (new_allocated >> 1) + 8; - if ( REALLOC_ARRAY( buffer->positions, new_allocated, HB_PositionRec ) ) - return error; + if ( buffer->positions ) + { + if ( REALLOC_ARRAY( buffer->positions, new_allocated, HB_PositionRec ) ) + return error; + } + if ( REALLOC_ARRAY( buffer->in_string, new_allocated, HB_GlyphItemRec ) ) return error; + if ( buffer->inplace ) { buffer->out_string = buffer->in_string; @@ -121,6 +126,22 @@ hb_buffer_new( HB_Buffer *buffer ) return HB_Err_Ok; } +HB_Error +hb_buffer_clear_positions( HB_Buffer buffer ) +{ + if ( !buffer->positions ) + { + HB_Error error; + + if ( ALLOC_ARRAY( buffer->positions, buffer->allocated, HB_PositionRec ) ) + return error; + } + + memset (buffer->positions, 0, sizeof (buffer->positions[0]) * buffer->in_length); + + return HB_Err_Ok; +} + void hb_buffer_clear_output( HB_Buffer buffer ) { diff --git a/src/harfbuzz-buffer.h b/src/harfbuzz-buffer.h index 9adfc5736..c0b047fa0 100644 --- a/src/harfbuzz-buffer.h +++ b/src/harfbuzz-buffer.h @@ -78,6 +78,9 @@ hb_buffer_clear( HB_Buffer buffer ); void hb_buffer_clear_output( HB_Buffer buffer ); +HB_Error +hb_buffer_clear_positions( HB_Buffer buffer ); + HB_Error hb_buffer_add_glyph( HB_Buffer buffer, FT_UInt glyph_index, diff --git a/src/harfbuzz-gpos.c b/src/harfbuzz-gpos.c index 5c91f6eaf..52ed12065 100644 --- a/src/harfbuzz-gpos.c +++ b/src/harfbuzz-gpos.c @@ -6091,7 +6091,7 @@ HB_Error HB_GPOS_Apply_String( FT_Face face, { HB_Error error, retError = HB_Err_Not_Covered; GPOS_Instance gpi; - FT_UShort i, j, lookup_count; + int i, j, lookup_count, num_features; if ( !face || !gpos || !buffer ) return HB_Err_Invalid_Argument; @@ -6106,13 +6106,16 @@ HB_Error HB_GPOS_Apply_String( FT_Face face, gpi.dvi = dvi; lookup_count = gpos->LookupList.LookupCount; + num_features = gpos->FeatureList.ApplyCount; - if ( gpos->FeatureList.ApplyCount ) + if ( num_features ) { - memset (buffer->positions, 0, sizeof (buffer->positions[0]) * buffer->in_length); + error = hb_buffer_clear_positions( buffer ); + if ( error ) + return error; } - for ( i = 0; i < gpos->FeatureList.ApplyCount; i++ ) + for ( i = 0; i < num_features; i++ ) { FT_UShort feature_index = gpos->FeatureList.ApplyOrder[i]; HB_Feature feature = gpos->FeatureList.FeatureRecord[feature_index].Feature; @@ -6136,7 +6139,7 @@ HB_Error HB_GPOS_Apply_String( FT_Face face, } } - if ( gpos->FeatureList.ApplyCount ) + if ( num_features ) { error = Position_CursiveChain ( buffer ); if ( error ) diff --git a/src/harfbuzz-gsub.c b/src/harfbuzz-gsub.c index 16a53850b..3f70e0d0f 100644 --- a/src/harfbuzz-gsub.c +++ b/src/harfbuzz-gsub.c @@ -4319,7 +4319,7 @@ HB_Error HB_GSUB_Apply_String( HB_GSUBHeader* gsub, HB_Buffer buffer ) { HB_Error error, retError = HB_Err_Not_Covered; - FT_UShort i, j, lookup_count; + int i, j, lookup_count, num_features; if ( !gsub || !buffer) @@ -4329,8 +4329,9 @@ HB_Error HB_GSUB_Apply_String( HB_GSUBHeader* gsub, return retError; lookup_count = gsub->LookupList.LookupCount; + num_features = gsub->FeatureList.ApplyCount; - for ( i = 0; i < gsub->FeatureList.ApplyCount; i++) + for ( i = 0; i < num_features; i++) { FT_UShort feature_index = gsub->FeatureList.ApplyOrder[i]; HB_Feature feature = gsub->FeatureList.FeatureRecord[feature_index].Feature;