D16579: Remove support for non-standard APE tag field names from the test files

Stefan BrĂ¼ns noreply at phabricator.kde.org
Mon Mar 11 13:33:51 GMT 2019


bruns added a comment.


  The change, as you pushed it, was never open for review, so there was not even the chance to review.
  
  The new test files change the tested tag from one allowed variant to another. Both variants exist in the wild, and you changed the test coverage to the one you seemed to be more important. A good change would have increased the coverage.

REPOSITORY
  R286 KFileMetaData

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

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


More information about the Kde-frameworks-devel mailing list