From: Enlightenment S. <no-...@en...> - 2011-04-26 14:20:44
|
Log: Evas font-engine: Updated comments. Author: tasn Date: 2011-04-26 07:20:38 -0700 (Tue, 26 Apr 2011) New Revision: 58934 Trac: http://trac.enlightenment.org/e/changeset/58934 Modified: trunk/evas/src/lib/engines/common/evas_text_utils.c Modified: trunk/evas/src/lib/engines/common/evas_text_utils.c =================================================================== --- trunk/evas/src/lib/engines/common/evas_text_utils.c 2011-04-26 14:05:01 UTC (rev 58933) +++ trunk/evas/src/lib/engines/common/evas_text_utils.c 2011-04-26 14:20:38 UTC (rev 58934) @@ -70,7 +70,9 @@ } } -/* Won't work in the middle of ligatures, assumes cutoff < len */ +/* Won't work in the middle of ligatures, assumes cutoff < len. + * Also won't work in the middle of indic words, should handle that in a + * smart way. */ EAPI void evas_common_text_props_split(Evas_Text_Props *base, Evas_Text_Props *ext, int _cutoff) @@ -86,7 +88,7 @@ size_t i; itr = base->info->ot + base->start; _cutoff += base->text_offset; - /* FIXME: can I binary search? I don't think this is always sorted */ + /* Must do a linear search because this is not always sorted. */ for (i = 0 ; i < base->len ; i++, itr++) { if (itr->source_cluster == (size_t) _cutoff) |