D15714: add a string suffix to test data and use for unicode testing of taglibwriter

Stefan Brüns noreply at phabricator.kde.org
Tue Oct 9 23:12:25 BST 2018


bruns accepted this revision.
bruns added a comment.
This revision is now accepted and ready to land.


  Otherwise good to go ...

INLINE COMMENTS

> taglibwritertest.cpp:75
> +    // source encoding: "€µ"
> +    static const QChar data[4] = { 0x20ac, 0xb5 };
> +    QString unicodeTestStringSuffix(data, 2);

can you change this to `data[2]` ...
Facepalm myself ...

REPOSITORY
  R286 KFileMetaData

BRANCH
  taglib_write_unicode

REVISION DETAIL
  https://phabricator.kde.org/D15714

To: astippich, mgallien, bruns
Cc: smithjd, svuorela, kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181009/3650f7f1/attachment.html>


More information about the Kde-frameworks-devel mailing list