diff options
author | Konstantin Ritt <ritt.ks@gmail.com> | 2013-08-13 04:48:05 +0300 |
---|---|---|
committer | The Qt Project <gerrit-noreply@qt-project.org> | 2013-08-15 20:47:47 +0200 |
commit | f3ffebe5f8e7a7cb08df46a9f511db829411d1ae (patch) | |
tree | 927e6fba25c9b0d26f0c978c3232bceb19b7df76 /src/corelib/tools/qunicodetools.cpp | |
parent | a15b56b0c1eae0d368c16be2d81bba11cb1faee7 (diff) |
initCharAttributes() micro-optimization
Change-Id: Id8e275c9b4ae0a9855b8ba8917824c79cde5a919
Reviewed-by: Lars Knoll <lars.knoll@digia.com>
Diffstat (limited to 'src/corelib/tools/qunicodetools.cpp')
-rw-r--r-- | src/corelib/tools/qunicodetools.cpp | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/src/corelib/tools/qunicodetools.cpp b/src/corelib/tools/qunicodetools.cpp index 4d5c978fd5..b3e55a5abc 100644 --- a/src/corelib/tools/qunicodetools.cpp +++ b/src/corelib/tools/qunicodetools.cpp @@ -603,28 +603,35 @@ Q_CORE_EXPORT void initCharAttributes(const ushort *string, int length, if (options & WhiteSpaces) getWhiteSpaces(string, length, attributes); - if (!items || numItems <= 0) - return; if (!qt_initcharattributes_default_algorithm_only) { + if (!items || numItems <= 0) + return; + QVarLengthArray<HB_ScriptItem, 64> scriptItems; scriptItems.reserve(numItems); int start = 0; + HB_Script startScript = script_to_hbscript(items[start].script); + if (Q_UNLIKELY(startScript == HB_Script_Inherited)) + startScript = HB_Script_Common; for (int i = start + 1; i < numItems; ++i) { - if (script_to_hbscript(items[i].script) == script_to_hbscript(items[start].script)) + HB_Script script = script_to_hbscript(items[i].script); + if (Q_LIKELY(script == startScript || script == HB_Script_Inherited)) continue; + Q_ASSERT(items[i].position > items[start].position); HB_ScriptItem item; item.pos = items[start].position; item.length = items[i].position - items[start].position; - item.script = script_to_hbscript(items[start].script); + item.script = startScript; item.bidiLevel = 0; // unused scriptItems.append(item); start = i; + startScript = script; } if (items[start].position + 1 < length) { HB_ScriptItem item; item.pos = items[start].position; item.length = length - items[start].position; - item.script = script_to_hbscript(items[start].script); + item.script = startScript; item.bidiLevel = 0; // unused scriptItems.append(item); } |