D15068: Bindings: Correct handling of sources containing utf-8
Luca Beltrame
noreply at phabricator.kde.org
Fri Sep 14 13:21:41 BST 2018
lbeltrame accepted this revision.
lbeltrame added a comment.
This revision is now accepted and ready to land.
LGTM.
INLINE COMMENTS
> sip_generator.py:752
> #
> - return "".join(extract).replace("\n", " ")
> + return b''.join(extract).decode('utf-8').replace("\r\n", " ").replace("\n", " ").replace("\r", " ")
>
Perhaps mention in the comment that you're doing this for all the types of newlines?
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D15068
To: bruns, #frameworks, lbeltrame
Cc: lbeltrame, bcooksley, jtamate, kde-frameworks-devel, kde-buildsystem, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20180914/05071317/attachment.html>
More information about the Kde-buildsystem
mailing list