| 524d294be051584e5506e5a4ad6ec70471bf4c08 |
|
12-Dec-2017 |
Seigo Nonaka <nona@google.com> |
Stop creating local vector for storing temporary hyphenation result
No temporary vector is necessary during hypheanation.
At the same time, this reveals Range name confliction. Remove internal
Range struct and use public one in CmapCoverage.cpp
Bug: 65024629
Test: minikin_tests
Change-Id: I31fb3472bb6a88e4e15d851723f34adc25e1d993
/frameworks/minikin/tests/perftests/Hyphenator.cpp
|
| 6c8722e217ff5238f0b849152d7936959a728103 |
|
30-Nov-2017 |
Seigo Nonaka <nona@google.com> |
Apply clang-format
Bug: 65125938
Test: m
Change-Id: I4a4319cc34c186aa0a3ce1d0301af1cd4e0feb81
/frameworks/minikin/tests/perftests/Hyphenator.cpp
|
| 1d461589869ee5b7102f96271b0ef0a776ab513c |
|
20-Oct-2017 |
Seigo Nonaka <nona@google.com> |
Style cleanup.
This CL contains following cleanups:
- Rename the following classes and corresponding file names:
FontLanguage -> Locale
FontLanguages -> LocaleList
FontLanguageListCache -> LocaleListCache
- Remove unnecessary #include lines.
- Define the Minikin's header include ordering and follow that style.
See doc/minikin_style.md for details.
- Always use std::string and std::vector instead of string and vector
with using-declaration.
- Replace LOG_ALWAYS_FATAL_IF with MINIKIN_ASSERT so that the assertion
is removed on build configuration for user.
This CL does not change any behavior.
Bug: 65024629
Test: make checkbuild
Change-Id: If41bb3c3389303053bbe95e349eb2dd4b619257c
/frameworks/minikin/tests/perftests/Hyphenator.cpp
|
| b76fd0c864c1fc06eb25f214851b5f40e9c6a321 |
|
12-Oct-2017 |
Seigo Nonaka <nona@google.com> |
Use minikin's HyphenatorMap in LineBreaker.
This CL contains the following changes:
- Use language ID as the locale input of addStyleRun.
- Resolve Hyphenator with minikin's HyphenatorMap.
- Pool the ICU's LineBreaker.
Creating an ICU LineBreaker takes some time. Thus, pool
them and use the same instances later.
- Use FontLanguage as the locale input of WordBreaker.
Here is a performance scores: (w/o patch -> w/ patch)
StaticLayoutPerfTest (median):
createRandom: 4,044,875 -> 3,896,743 (-3.7%)
createRandom Balanced: 3,985,204 -> 3,821,016 (-4.1%)
TextViewOnMeasurePerfTest (median):
measure_AtMost: 44,064,070 -> 38,746,998 (-12.1%)
measure_Exactly: 43,028,018 -> 37,827,794 (-12.1%)
measure_Unspecified: 71,863,028 -> 65,774,110 (-8.5%)
Bug: 65024629
Bug: 67319341
Test: minikin_test
Change-Id: Ibf173e0b0b29c33eaad790b65dad6fd8a1190e47
/frameworks/minikin/tests/perftests/Hyphenator.cpp
|
| 5aa870f7ccd5138af60c96ec232192b52f967530 |
|
01-Sep-2017 |
Seigo Nonaka <nona@google.com> |
Remove mLocale and save locale into Hyphenator.
Save locale to Hyphenator so that LineBreaker would no longer need to
track the locales to be used for hyphenation separately.
Bug: 65024629
Test: bit CtsTextTestCases:android.text.cts.StaticLayoutTest
Test: bit CtsTextTestCases:android.text.cts.DynamicLayoutTest
Test: bit CtsWidgetTestCases:android.widget.cts.TextViewTest
Change-Id: I688624b1984f7e9111bed6df106f41828ad3be0f
/frameworks/minikin/tests/perftests/Hyphenator.cpp
|
| 2bfc56ad2fcd0f65551a9c2ed8a9c2350f2e0cd1 |
|
25-Apr-2017 |
Colin Cross <ccross@android.com> |
Convert frameworks/minikin/tests to Android.bp
See build/soong/README.md for more information.
Test: m -j checkbuild
Change-Id: I930debdd129da7f61ac4b764980f73dfd487785d
/frameworks/minikin/tests/perftests/Hyphenator.cpp
|
| d78f260a988024b878909555edbfcd7159e7ad2f |
|
28-Feb-2017 |
Roozbeh Pournader <roozbeh@google.com> |
Customizable min suffix/prefix length for hyphenation in Minikin
With this change, different languages can have a different minimum
length for suffix and prefixes when hyphenating. Previously, the
defaults used for English, 2 and 3, were used for every language.
Bug: 35712376
Test: Minikin unit tests were updated and the pass
Change-Id: Iffaf11c6b208c57d28d45b17246e177572dc1210
/frameworks/minikin/tests/perftests/Hyphenator.cpp
|
| c7ef4000c1e840c3d3b66e85a40ebd34a5a2a8ee |
|
18-Feb-2017 |
Roozbeh Pournader <roozbeh@google.com> |
Correct hyphenation for various complex cases
This adds better support for Arabic script languages, Armenian,
Catalan, Hebrew, Kannada, Malayalam, Polish, Tamil, and Telugu by
adding various hyphenation types and edits appropriate for the
locales.
For Arabic script languages, soft hyphens act transparently with
regard to joining: If a line is broken at a soft hyphen where the two
characters around the soft hyphen were joining each other before,
they will continue to appear joining if the line is broken at the
soft hyphen and a hyphen glyph is inserted. This is needed for
Central Asian languages such as Uighur.
For Armenian, U+058A ARMENIAN HYPHEN is used for line breaks caused
by either automatic hyphenation or soft hyphens.
For Catalan, nonstandard line breaks are implemented for "l·l", which
hyphenates as "l-/l".
For Polish, when there is a line break at a hyphen, the hyphen is
repeated at the next line.
For the South Indic languages, when breaks happen due to soft breaks
or automatic hyphenation, no visible hyphen is inserted, although a
penalty is added.
For Hebrew, support for using U+05BE HEBREW PUNCTUATION MAQAF has
been implemented, but it's turned off pending confirmation of
desirability.
Also, hard hyphens, which previously had no penalty added for
breaking the line after them, now have the same penalty as an
automatic or soft break, with the difference that no hyphen is
inserted when they break.
Finally, some bugs have been fixed with hyphenating multiscript and
multi-font words.
Bug: 19950445
Bug: 19955011
Bug: 25623243
Bug: 26154469
Bug: 26154471
Bug: 33387871
Bug: 33560754
Bug: 33752592
Bug: 33754204
Test: Unit tests added, plus thorough manual testing
Change-Id: Iaccf776ce8d1d434ee8b1c534ff3659d80fdc338
/frameworks/minikin/tests/perftests/Hyphenator.cpp
|
| 0ca4fb6d44160245ad2333851ac18a13fc553ec9 |
|
14-Jun-2016 |
Seigo Nonaka <nona@google.com> |
Add more native perf tests to minikin.
This CL introduces performance tests for following three modules:
- Hyphenator
- WordBreaker
- GraphemeBreak
During using UnicodeUtils, need to decouple it from gtest since
perftest doesn't have gtest dependencies.
Bug:29142734
Change-Id: I700c662fa7d0a52f19d8e93150ad1a85dc28769f
/frameworks/minikin/tests/perftests/Hyphenator.cpp
|