D18915: Fix batchrename changing extension to lower case

Christoph Feck noreply at phabricator.kde.org
Fri Mar 8 20:52:00 GMT 2019


cfeck added inline comments.

INLINE COMMENTS

> bruns wrote in batchrenamejob.cpp:165
> `QString extension = GetFileExtension(url.fileName());`
> ...
> `static QString BatchRenameJobPrivate::GetFileExtension(QString filename)`

Function/method names are usually lowercase. Also, we don't add `get` for getters, only `set` for setters.

⇒ `fileExtension()` ?

Additionally, we pass QString via reference

⇒ `QString &filename` ?

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: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190308/a4463517/attachment.html>


More information about the Kde-frameworks-devel mailing list