<table><tr><td style="">svuorela added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D20938">View Revision</a></tr></table><br /><div><div><p>After reading the code a bit, I wonder</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">can't some of the parsing code be made testable (and tests added)<ul class="remarkup-list">
<li class="remarkup-list-item">This likely also requires to get rid of the global static.</li>
</ul></li>
<li class="remarkup-list-item">rather than poll /proc/mounts every second, I was wondering if one of the filesystem notification things (QFileSystemWatcher, KDirWatch) could tell when to re-parse it</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R245 Solid</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20938">https://phabricator.kde.org/D20938</a></div></div><br /><div><strong>To: </strong>hallas, Frameworks, ngraham, elvisangelaccio, broulik, bruns<br /><strong>Cc: </strong>svuorela, nicolasfella, ivan, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>