mgallien added a comment.
In D10694#224465 <https://phabricator.kde.org/D10694#224465>, @michaelh wrote: > In D10694#224440 <https://phabricator.kde.org/D10694#224440>, @mgallien wrote: > > > Could you please update your diff and we can land it ? This is a useful improvement. > > > 1. We can't land it yet. baloo searching breaks with this patch. baloo has be adapted first. I trust you on this one but on some property the taglib extractor is already doing multiple adds of the same property. It means Baloo is already storing lists. I can help you but have not much time as usual. > 1. I don't know what to update. Please tell me once more what you want me to change. You can have a look at the taglib extractor. There are multiple examples of for loops adding multiple times the same property. > For this change concerted actions are needed. Let's discuss general questions here: T8196 <https://phabricator.kde.org/T8196> Thanks, this is a good idea. REPOSITORY R286 KFileMetaData REVISION DETAIL https://phabricator.kde.org/D10694 To: michaelh, mgallien, dfaure Cc: astippich, #frameworks, ashaposhnikov, michaelh, spoorun, navarromorales, isidorov, nicolasfella, firef, andrebarros, alexeymin, emmanuelp
