diff options
author | Eskil Abrahamsen Blomfeldt <eskil.abrahamsen-blomfeldt@nokia.com> | 2012-04-26 16:47:56 +0200 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2012-04-26 17:52:12 +0200 |
commit | f1212ddc62a919001fcd5aebcb37e63170c475fe (patch) | |
tree | b9630393e599d01ea96789b39d8b49b0127e4587 /src/3rdparty/harfbuzz | |
parent | 0432d550e85cf062fae69b8a83288897394b5365 (diff) |
Revert "Harfbuzz-shaper - fix incorrect logClusters being set in HB_OpenTypePosition"
This reverts commit add629d4f16a536fc56d55727195e3247b621a54.
The commit caused crashes when laying out some text (e.g. ordinary
Arabic text) with certain fonts (it triggered the assert in
QTextLayout::addNextCluster()). The regressions will have to be
weeded out before the fix can be recommitted.
Change-Id: Iee457f138367d4cf8fcbd2e518271d8eaa95a62c
Reviewed-by: Jiang Jiang <jiang.jiang@nokia.com>
Diffstat (limited to 'src/3rdparty/harfbuzz')
-rw-r--r-- | src/3rdparty/harfbuzz/src/harfbuzz-shaper.cpp | 48 |
1 files changed, 13 insertions, 35 deletions
diff --git a/src/3rdparty/harfbuzz/src/harfbuzz-shaper.cpp b/src/3rdparty/harfbuzz/src/harfbuzz-shaper.cpp index e630ec3808..f6900325bc 100644 --- a/src/3rdparty/harfbuzz/src/harfbuzz-shaper.cpp +++ b/src/3rdparty/harfbuzz/src/harfbuzz-shaper.cpp @@ -1302,52 +1302,30 @@ HB_Bool HB_OpenTypePosition(HB_ShaperItem *item, int availableGlyphs, HB_Bool do glyphs[i] = face->buffer->in_string[i].gindex; attributes[i] = face->tmpAttributes[face->buffer->in_string[i].cluster]; if (i && face->buffer->in_string[i].cluster == face->buffer->in_string[i-1].cluster) - attributes[i].clusterStart = false; //FIXME - Shouldn't we otherwise set this to true, rather than leaving it? + attributes[i].clusterStart = false; } item->num_glyphs = face->buffer->in_length; if (doLogClusters && face->glyphs_substituted) { // we can't do this for indic, as we pass the stuf in syllables and it's easier to do it in the shaper. - // #### the reconstruction of the logclusters currently does not work if the original string - // contains surrogate pairs - unsigned short *logClusters = item->log_clusters; int clusterStart = 0; - int oldIntermediateIndex = 0; - - // This code makes a mapping, logClusters, between the original utf16 string (item->string) and the final - // set of glyphs (in_string). - // - // The code sets the value of logClusters[i] to the index of in_string containing the glyph that will render - // item->string[i]. - // - // This is complicated slightly because in_string[i].cluster is an index to an intermediate - // array of glyphs - the array that we were passed as the original value of item->glyphs. - // To map from the original string to the intermediate array of glyphs we have tmpLogClusters. - // - // So we have three groups of indexes: - // - // i,clusterStart = index to in_length, the final set of glyphs. Also an index to attributes - // intermediateIndex = index to the glyphs originally passed in. - // stringIndex = index to item->string, the original string. - - int stringIndex = 0; - // Iterate over the final set of glyphs... + int oldCi = 0; + // #### the reconstruction of the logclusters currently does not work if the original string + // contains surrogate pairs for (unsigned int i = 0; i < face->buffer->in_length; ++i) { - // Get the index into the intermediate string for the start of the cluster of chars - int intermediateIndex = face->buffer->in_string[i].cluster; - if (intermediateIndex != oldIntermediateIndex) { - // We have found the end of the cluster of chars in the intermediate string - while (face->tmpLogClusters[stringIndex] < intermediateIndex) { - logClusters[stringIndex++] = clusterStart; - } + int ci = face->buffer->in_string[i].cluster; + // DEBUG(" ci[%d] = %d mark=%d, cmb=%d, cs=%d", + // i, ci, glyphAttributes[i].mark, glyphAttributes[i].combiningClass, glyphAttributes[i].clusterStart); + if (!attributes[i].mark && attributes[i].clusterStart && ci != oldCi) { + for (int j = oldCi; j < ci; j++) + logClusters[j] = clusterStart; clusterStart = i; - oldIntermediateIndex = intermediateIndex; + oldCi = ci; } } - while (stringIndex < face->length) { - logClusters[stringIndex++] = clusterStart; - } + for (int j = oldCi; j < face->length; j++) + logClusters[j] = clusterStart; } // calulate the advances for the shaped glyphs |