transifex / transifex-java

Transifex Native Toolkit for Java
Apache License 2.0
19 stars 3 forks source link

Improve spannable support performance #44

Closed Petrakeas closed 2 years ago

Petrakeas commented 2 years ago

Spanable support is now enabled by default. The documentation and readme have been updated accordingly. An affected unit test has been updated.

When NativeCore#setSupportSpannable(true) was called, strings were processed by fromHTML() even when they contained no tags. Since fromHTML() has a performance cost, we had a perfomrance penalty even when we didn't need it.

With this update, we apply fromHTML() only in strings that contain a "<" character and hence probably contain tags. This way, the performance penalty occurs only when needed.