D23010: Use range loop if possible

Christoph Cullmann noreply at phabricator.kde.org
Wed Aug 7 20:54:06 BST 2019


cullmann added inline comments.

INLINE COMMENTS

> gawin wrote in katefilebrowserconfig.cpp:162
> I'm not sure about name. Maybe `actionName` would be better?

Why not ;=)

> gawin wrote in katemdi.cpp:983
> Also, camel case?

If you like, have no issues with both variants. But better qAsConst(m_toolViews)

REPOSITORY
  R40 Kate

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

To: gawin, #kate, cullmann
Cc: cullmann, kwrite-devel, #kate, univerz, domson, michaelh, ngraham, demsking, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20190807/608aa554/attachment-0001.html>


More information about the KWrite-Devel mailing list