ngraham added inline comments.

INLINE COMMENTS

> configcommand.cpp:43
> +{
> +    while (path.endsWith(QLatin1Char('/'))) {
> +        path.chop(1);

Does this need to be a `while`? Why not a regular old `if`? Are you trying to 
handle the case where it ends with multiple trailing slashes?

REPOSITORY
  R293 Baloo

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

To: bruns, #baloo, #frameworks, poboiko, ngraham
Cc: kde-frameworks-devel, ashaposhnikov, michaelh, astippich, spoorun, ngraham, 
bruns, abrahams

Reply via email to