D17193: KateViewInternal: Remove unneeded functions

2018-11-29 Thread Christoph Cullmann
This revision was automatically updated to reflect the committed changes. Closed by commit R39:8b4cd5f37947: KateViewInternal: Remove unneeded functions (authored by loh.tar, committed by cullmann). REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17193?vs=46

D17193: KateViewInternal: Remove unneeded functions

2018-11-29 Thread Christoph Cullmann
cullmann accepted this revision. cullmann added a comment. This revision is now accepted and ready to land. I am happy with this ;=) Thanks for adding documentation to the header, such stuff is very welcome! REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D17193

D17193: KateViewInternal: Remove unneeded functions

2018-11-29 Thread loh tar
loh.tar updated this revision to Diff 46473. loh.tar edited the summary of this revision. loh.tar edited the test plan for this revision. loh.tar added a comment. - Prefer function calls for member excess - Merge KateViewInternal::updateView with KateViewInternal::doUpdateView - Add documen

D17193: KateViewInternal: Remove unneeded functions

2018-11-28 Thread Christoph Cullmann
cullmann requested changes to this revision. cullmann added a comment. This revision now requires changes to proceed. I would prefer function calls to members, in the most cases, the performance difference is not measurable, even without inline. For non-exported functions that are one lin

D17193: KateViewInternal: Remove unneeded functions

2018-11-28 Thread loh tar
loh.tar updated this revision to Diff 46411. loh.tar added a comment. I think that's all. Well, it looks less beneficial as previous anticipated by me, but I think it's still the right direction. Just a view questions regarding the mix in the KTextEditor::ViewPrivate code how to acces

D17193: KateViewInternal: Remove unneeded functions

2018-11-27 Thread Christoph Cullmann
cullmann added a comment. Simplifications in that way are ok in my eye, given the extra level of functions have no gain. (not even encapsulation) REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D17193 To: loh.tar, #ktexteditor Cc: cullmann, kwrite-devel, kde-frame

D17193: KateViewInternal: Remove unneeded functions

2018-11-27 Thread loh tar
loh.tar created this revision. loh.tar added a reviewer: KTextEditor. Herald added projects: Kate, Frameworks. Herald added subscribers: kde-frameworks-devel, kwrite-devel. loh.tar requested review of this revision. TEST PLAN This is only an example snippet to show that obviously something could