michaelh added a comment.

  I went I little overboard with `remove_if` and had to repair.
  Device filtering is also broken see inline comment.

INLINE COMMENTS

> databasesanitizer.cpp:160
>                  const QByteArray rootPath = 
> QFile::encodeName(vol.rootPath());
>                  const auto fsinfo = filePathToStat(rootPath);
>                  const quint32 id = static_cast<quint32>(fsinfo.st_dev);

$ baloodb list --mounted-only Einstein
  Listing database contents...
  ! device: 43 inode: 319 url: /media/circulans/(tv-analog)/L-Z/Was Einstein 
noch nicht wusste.mkv
  + device: 46 inode: 319 url: /media/circulans/(doku)/Was Einstein noch nicht 
wusste.mkv

43 is the correct id for a share. The path is wrong.
46 is a wrong id for the same share. It should not be listed. The path however 
is correct.

`QStorageInfo` is not enough, I'm afraid. For proper matching mtab must be read.

REPOSITORY
  R293 Baloo

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

To: michaelh, #baloo, #frameworks
Cc: bruns, ashaposhnikov, michaelh, astippich, spoorun

Reply via email to