[PATCH] Use UTF-8 for text strings.
Thiago Macieira
thiago at macieira.org
Mon Dec 16 14:18:18 UTC 2013
On segunda-feira, 16 de dezembro de 2013 22:49:03, Michael Andreen wrote:
> Fixes #371
>
> Signed-off-by: Michael Andreen <harv at ruin.nu>
While the patch itself is fine, I'm not sure it's actually fixing anything.
Michael, can you confirm that it does fix the problem?
The reason being that we set the "codec for C strings" early in the
initialisation, so
QString(text) == QString::fromUtf8(text)
[though with a different codec implementation and a small performance penalty.
(I need to go fix the different codec problem for Qt 5, now that I've improved
the performance of our UTF-8 codec 2.1x to 3.6x times on average data over
ICU[1])]
[1] https://codereview.qt-project.org/69597
--
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
Software Architect - Intel Open Source Technology Center
PGP/GPG: 0x6EF45358; fingerprint:
E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20131216/48efadff/attachment.sig>
More information about the subsurface
mailing list