On quarta-feira, 2 de maio de 2012 16.26.42, Thiago Macieira wrote:
> The following two pending commit changes the QString 8-bit functions to use
> UTF-8, by *temporarily* changing fromAscii to mean fromUtf8, and toAscii to
> mean toUtf8.
>       https://codereview.qt-project.org/24700
>       https://codereview.qt-project.org/24701
>       tests: https://codereview.qt-project.org/24702

They're in.

> They have been tested in qtbase and no regressions have been found. I do not
> believe they should cause regressions in other modules.
>
> I'm now testing a series of changes that change fromAscii to fromUtf8, as
> well as correct one or two encoding mistakes I think I've found. Since
> fromAscii == fromUtf8 at this point in the test, the change is technically
> a no-op and I expect no regressions at all. Those changes are done for the
> few places in the code where the data seemed to be non-Latin1 in origin, as
> well as QString itself.

Those are done.

> Next, I'll change all remaining fromAscii to fromLatin1 and toAscii to
> toLatin1. Since that's what those functions were before (still are right now
> in qtbase master), I also expect no regressions. Then I'll deprecate the
> Ascii functions.

Done, except for one change in qmake that is still not approved.

> Finally, probably starting two weeks from now when I'm back from the US,
> I'll start benchmarking and optimising the fromUtf8 function, as well as
> merging the many UTF-8 encoders and decoders in Qt (yes, we have more than
> one).

That's the next step.

QString's constructor is now UTF-8. The change has just been approved by the
CI.

--
Thiago Macieira - thiago.macieira (AT) intel.com
  Software Architect - Intel Open Source Technology Center
     Intel Sweden AB - Registration Number: 556189-6027
     Knarrarnäsgatan 15, 164 40 Kista, Stockholm, Sweden

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
Development mailing list
[email protected]
http://lists.qt-project.org/mailman/listinfo/development

Reply via email to