michaelh marked an inline comment as done.
michaelh added inline comments.

INLINE COMMENTS

> databasesanitizer.cpp:196
> +                              summary.accessible += (info.deviceId == 
> deviceId && info.accessible ? 1 : 0);
> +                              return info.deviceId == deviceId;
> +                          });

~3 hrs for this, phew.

> michaelh wrote in databasesanitizer.cpp:160
>   $ 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.

On the wrong track

REPOSITORY
  R293 Baloo

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

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

Reply via email to