bruns accepted this revision.
bruns added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> fstabhandling.cpp:246
> +        if (deviceName.endsWith(QLatin1Char('/'))) {
> +            // remove trailing slash
> +            deviceName.chop(1);

This comment is stating the obvious ...

> fstabhandling.cpp:249
> +        } else {
> +            // add a trailing slash
> +            deviceName.append(QLatin1Char('/'));

dito

REPOSITORY
  R245 Solid

BRANCH
  arcpatch-D21204

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

To: meven, bruns, #frameworks, ngraham
Cc: broulik, dhaumann, anthonyfieroni, ngraham, bruns, apol, 
kde-frameworks-devel, LeGast00n, GB_2, michaelh

Reply via email to