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