<table><tr><td style="">cfeck added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D11552">View Revision</a></tr></table><br /><div><div><p>I have zero knowledge about Baloo, but I can add some comments regarding Unicode.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">the four ranges you used are all adjacent, so you could contract to {0x4E00, 0x9FFF}</li>
<li class="remarkup-list-item">there are more ranges for CJK characters in the BMP, at least {0x3400, 0x4DBF} would be useful (I don't know if CJK users ever use the compatibility characters)</li>
<li class="remarkup-list-item">to be able to fully support the remaining CJK blocks in higher planes, the could would need to handle surrogate pairs</li>
<li class="remarkup-list-item">if Baloo doesn't handle CJK, it maybe also doesn't handle other non-Latin scripts, so I suggest to use QChar::category()</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R293 Baloo</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11552">https://phabricator.kde.org/D11552</a></div></div><br /><div><strong>To: </strong>michaelh, Baloo, Frameworks, lbeltrame, bruns<br /><strong>Cc: </strong>cfeck, ashaposhnikov, michaelh, astippich, spoorun, nicolasfella, ngraham, alexeymin<br /></div>