Set the whiteSpace flag outside the grapheme and the line breaking loop
authorKonstantin Ritt <ritt.ks@gmail.com>
Tue, 5 Jun 2012 08:19:22 +0000 (11:19 +0300)
committerQt by Nokia <qt-info@nokia.com>
Thu, 7 Jun 2012 19:18:36 +0000 (21:18 +0200)
commit824180a12249e48c0e3280fec64940825ce0aa6e
tree36dba52c02f1603551ef2820e788796c47b5fb38
parent9d3e77f18e3bd19c1723199f0dfb5af9eabdabe8
Set the whiteSpace flag outside the grapheme and the line breaking loop

The white spaces determination doesn't belong to the text breaking algorithm.
A proper breaking implementation shouldn't assume spaces are
break opportunities (actually, space is allowed to be a grapheme base);
However, the whiteSpace flag should never be checked alone while iterating
over the text to find the space sequence; the grapheme boundaries should always
be taken into account. This covers the SMP code points in UTF-16 text and
graphemes that consist of a space followed with one or more grapheme extenders.

This introduces a minor overhead that would be eliminated some later.

Change-Id: Ic2cc7f485631fd0b436fc256ce112ded5f94fc07
Reviewed-by: Lars Knoll <lars.knoll@nokia.com>
src/corelib/tools/qunicodetools.cpp