The Birdfont Source Code


All Repositories / birdfont.git / commit – RSS feed

Solve the backtrack issue

These changes was commited to the Birdfont repository Mon, 12 Oct 2015 15:05:32 +0000.

Contributing

Send patches or pull requests to johan.mattsson.m@gmail.com.
Clone this repository: git clone https://github.com/johanmattssonm/birdfont.git
author Johan Mattsson <johan.mattsson.m@gmail.com>
Mon, 12 Oct 2015 15:05:32 +0000 (17:05 +0200)
committer Johan Mattsson <johan.mattsson.m@gmail.com>
Mon, 12 Oct 2015 15:05:32 +0000 (17:05 +0200)
commit f0d8ed31ad94cfa9a38a69b76b69366b56152939
tree 6e35ac5f44ead0be5ae878714c616905e1c6be70
parent cf52bf0f58c63cc029819bed802abc2e88c8b5e2
Solve the backtrack issue

libbirdfont/OpenFontFormat/ContextualLigature.vala
--- a/libbirdfont/OpenFontFormat/ContextualLigature.vala +++ b/libbirdfont/OpenFontFormat/ContextualLigature.vala @@ -46,14 +46,7 @@ public FontData get_font_data (GlyfTable glyf_table, uint16 ligature_lookup_index) throws GLib.Error { FontData fd = new FontData (); - Font font = BirdFont.get_current_font (); // FIXME: thread safety? - - // FIXME: it looks like get_names is the right function - // but harfbuzz assumes that glyphs appear in the other - // order for latin scripts, get_names_in_reverse_order - // creates an array of glyphs in reverse order. - // - // I have not found out why yet. + Font font = BirdFont.get_current_font (); Gee.ArrayList<string> backtrack = font.get_names_in_reverse_order (backtrack); Gee.ArrayList<string> input = font.get_names (input);