bruns added inline comments.

INLINE COMMENTS

> fstabhandling.cpp:168
>  
> +            // strips last slash
> +            if (mountpoint.at(mountpoint.length() -1) == '/') {

This is a litlle bit scarce ...

> apol wrote in fstabhandling.cpp:170
> you can call mountpoint.chop(2) for the same effect, which should be easier 
> to read.

and why "2"?

REPOSITORY
  R245 Solid

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

To: meven
Cc: bruns, apol, kde-frameworks-devel, michaelh, ngraham

Reply via email to