hallas added a comment.

  In D20938#459218 <https://phabricator.kde.org/D20938#459218>, @bruns wrote:
  
  > Solid already has a working implementation for reading from /proc/mounts, 
the fstab backend.
  >
  > Contrary to this code, the fstab backend does not poll every second using a 
timer, but correctly uses /proc/mounts changes notification via blocking read.
  >
  > The only thing needed is a small extension to also hand out information for 
other filesystems than nfs and cifs.
  
  
  This was exactly the kind of review comments I was looking for :D Let me take 
a new stab at this where I extend the current fstab with this functionality so 
we can see how that solution compares to this one.

REPOSITORY
  R245 Solid

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

To: hallas, #frameworks, ngraham, elvisangelaccio, broulik, bruns
Cc: svuorela, nicolasfella, ivan, kde-frameworks-devel, michaelh, ngraham, bruns

Reply via email to