D18915: Fix batchrename changing extension to lower case

Stefan BrĂ¼ns noreply at phabricator.kde.org
Fri Mar 15 00:37:33 GMT 2019


bruns added a comment.


  Now, a unit test would be nice ...

INLINE COMMENTS

> batchrenamejob.cpp:177
> +    }
> +    return QStringLiteral();
> +}

This should be `return QString();`, wont get any cheaper than that.

REPOSITORY
  R241 KIO

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

To: cfoster, #dolphin, #frameworks, abalaji
Cc: cfeck, bruns, ngraham, elvisangelaccio, chinmoyr, kde-frameworks-devel, michaelh
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190315/8151101a/attachment.htm>


More information about the kfm-devel mailing list