apol added inline comments. INLINE COMMENTS
> fstabhandling.cpp:169 > + // strips last slash > + if (mountpoint.at(mountpoint.length() -1) == '/') { > + mountpoint = mountpoint.left(mountpoint.length() - 2); mountpoint.endsWith(QLatin1Char('/')) > fstabhandling.cpp:170 > + if (mountpoint.at(mountpoint.length() -1) == '/') { > + mountpoint = mountpoint.left(mountpoint.length() - 2); > + } you can call mountpoint.chop(2) for the same effect, which should be easier to read. > fstabhandling.cpp:217 > + // strips last slash > + if (mountpoint.at(mountpoint.length() -1) == '/') { > + mountpoint = mountpoint.left(mountpoint.length() - 2); Same as above. REPOSITORY R245 Solid REVISION DETAIL https://phabricator.kde.org/D21204 To: meven Cc: apol, kde-frameworks-devel, michaelh, ngraham, bruns