D18826: Rewrite the taglib extractor to use the generic PropertyMap interface
James Smith
noreply at phabricator.kde.org
Mon Feb 25 04:00:47 GMT 2019
smithjd added inline comments.
INLINE COMMENTS
> taglibextractor.cpp:109
> +{
> + if (savedProperties.contains("TITLE")) {
> + result->add(Property::Title, TStringToQString(savedProperties["TITLE"].toString()).trimmed());
This could return early if the property map is empty.
> taglibextractor.cpp:195
> }
> -
> - itApe = lstApe.find("PERFORMER");
> - if (itApe != lstApe.end()) {
> - if (!data.performer.isEmpty()) {
> - data.performer += ", ";
> + /* Lyricist is called "WRITER" for wma/asf files */
> + if (savedProperties.contains("WRITER")) {
Can this be moved into the asf-specific extractions?
REPOSITORY
R286 KFileMetaData
REVISION DETAIL
https://phabricator.kde.org/D18826
To: astippich, ngraham, bruns, mgallien
Cc: smithjd, kde-frameworks-devel, #baloo, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190225/46cf5b86/attachment.html>
More information about the Kde-frameworks-devel
mailing list