Review Request 115540: Wrap string literals in QStringLiteral in headers so projects with QT_NO_CAST_FROM_ASCII can use them

Teo Mrnjavac teo at kde.org
Fri Feb 7 16:12:57 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115540/
-----------------------------------------------------------

Review request for KDE Frameworks.


Repository: qca


Description
-------

KDE frameworks are built with QT_NO_CAST_FROM_ASCII defined. Some of the QCA headers contain instances of implicit conversion of string literals into QString, and this makes it hard for frameworks to use QCA.
This proposed change wraps those instances in QStringLiteral (Qt5) or QString::fromUtf8 (Qt4).

Note: submitting this to kdeframeworks because there seems to be no qca group, and emailing the QCA maintainers.


Diffs
-----

  include/QtCrypto/qca_basic.h 100c626 
  include/QtCrypto/qcaprovider.h b3d40ce 

Diff: https://git.reviewboard.kde.org/r/115540/diff/


Testing
-------

Builds fine, KSecretsService links fine against it.


Thanks,

Teo Mrnjavac

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140207/bab86387/attachment.html>


More information about the Kde-frameworks-devel mailing list