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