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

Reply via email to