michaelh added inline comments.

INLINE COMMENTS

> taglibextractor.cpp:157
> +    if (itAlbumArtists != allTags.end()) {
> +        data.albumArtists = 
> itAlbumArtists->second.toStringList().toString(", ");
>      }

This is an example for what we're discussing. Why the join??? We might lose 
info, when the album artist contains a comma.

REPOSITORY
  R286 KFileMetaData

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

To: michaelh, mgallien, #baloo, #frameworks, astippich
Cc: astippich, ashaposhnikov, michaelh, spoorun, nicolasfella, ngraham, 
alexeymin

Reply via email to