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