diff options
author | Raph Levien <raph@google.com> | 2012-10-19 14:00:56 -0700 |
---|---|---|
committer | Raph Levien <raph@google.com> | 2012-10-19 14:29:09 -0700 |
commit | 6842c622d8b68251f7d696d22bfa44a16c0bef46 (patch) | |
tree | 8ae52f66b4a28d1e5f4876a3a383e986fbc62816 | |
parent | d124f9692dc8dad8f5f77c293fe6d4ec1a0c02ea (diff) | |
download | harfbuzz-6842c622d8b68251f7d696d22bfa44a16c0bef46.tar.gz |
Fix to make space width consistent in bidiandroid-sdk-support_r11android-cts-4.2_r2android-cts-4.2_r1android-4.2_r1android-4.2.2_r1.2android-4.2.2_r1.1android-4.2.2_r1android-4.2.1_r1.2android-4.2.1_r1.1android-4.2.1_r1tools_r22jb-mr1.1-releasejb-mr1.1-dev-plus-aospjb-mr1.1-devjb-mr1-releasejb-mr1-dev-plus-aospjb-mr1-dev
This is a fix for bug 7363916, where the inconsistent breaking of the
text into script runs was leading to a different measured width on paint
than on measure for a TextView, leading to the content being ellipsized.
This patch classifies the space as Latin, rather than the script of the
surrounding text, which means that it's always drawn from the Latin
font, and thus is measured consistently.
Change-Id: I07abfa2918698c93f55333f1e964cc24a18d3def
-rwxr-xr-x | contrib/harfbuzz-unicode.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/contrib/harfbuzz-unicode.c b/contrib/harfbuzz-unicode.c index 29c14b3..432d5b0 100755 --- a/contrib/harfbuzz-unicode.c +++ b/contrib/harfbuzz-unicode.c @@ -78,13 +78,6 @@ script_property_cmp(const void *vkey, const void *vcandidate) { HB_Script code_point_to_script(uint32_t cp) { - /* BEGIN android-changed - For the purpose of aggregating script runs together, we treat space - as belonging to the same script as surrounding characters. This is a - performance optimization to keep the number of runs down. */ - if (cp == ' ') return HB_Script_Inherited; - /* END android-changed */ - const void *vprop = bsearch((void *) (intptr_t) cp, script_properties, script_properties_count, sizeof(struct script_property), |