[X2Go-Dev] Patch to support Qt5 as well as Qt4
Mihai Moldovan
ionic at ionic.de
Fri Apr 1 06:36:25 CEST 2016
On 21.03.2016 03:13 PM, Sébastien Ducoulombier wrote:
> Attached is a patch for X2Go Client so to support Qt5.
Thanks again for your patch!
The only problem I see with it is that all instances of QString::toAscii () have
been replaced by QString::toUtf8 (). Personally, I like the idea of porting
things to UTF-8, but fear side-effects in terms of passing UTF-8-encoded strings
to other interfaces that actually expect ASCII (or really a plain 8bit encoding.)
Hence, I'd like to modify all these occurrences with QString::toLatin1 () for
now. At the same time, I think your original patch serves as a great reference
as to what locations can potentially be rather ported to using UTF-8. I'll need
to examine where this is actually possible more accurately at a later time.
Does this make sense to you? Are there any objections?
Mihai
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 884 bytes
Desc: OpenPGP digital signature
URL: <http://lists.x2go.org/pipermail/x2go-dev/attachments/20160401/8ee2fd67/attachment.pgp>
More information about the x2go-dev
mailing list