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