meven marked 3 inline comments as done. meven added a comment.
In D28590#656297 <https://phabricator.kde.org/D28590#656297>, @bruns wrote: > In D28590#656133 <https://phabricator.kde.org/D28590#656133>, @meven wrote: > > > In D28590#654139 <https://phabricator.kde.org/D28590#654139>, @bruns wrote: > > > > > Do not create m_storageAccess in the constructor > > > > > > Hmm, you told me `the f (m_displayName.isEmpty()) block belongs here`, I don't see why instantiating `m_storageAccess` here is bad but mehh. > > > Please read again. > > 1. m_storageAccess is comparatively expensive > 2. I showed you how to get the mount state without StorageAccess > 3. I told you where to initialize it, yes. Doing it correctly of course still applies. Btw I updated already the code to use `FstabHandling::currentMountPoints` and `FstabHandling::currentMountPoints(m_device);` as you suggested (as StorageAccess does). REPOSITORY R245 Solid REVISION DETAIL https://phabricator.kde.org/D28590 To: meven, #frameworks, bruns, sitter Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns