This revision was automatically updated to reflect the committed changes.
Closed by commit R39:3f0c617e22ab: disconnect contextmenu from all
aboutToXXContextMenu receivers (authored by rjvbb).
CHANGED PRIOR TO COMMIT
https://phabricator.kde.org/D16927?vs=45597&id=45664#toc
REPOSITORY
R39 KTe
cullmann added a comment.
You are right. Please add some comment why one disconnects all things and
push it.
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D16927
To: rjvbb, #ktexteditor, #frameworks, cullmann
Cc: cullmann, kde-frameworks-devel, kwrite-devel, mic
rjvbb added a comment.
> Btw., the 2 nullptr in the disconnect can be left out, or?
Not to my understanding. From the docs:
Disconnect everything connected to a specific signal:
disconnect(myObject, SIGNAL(mySignal()), 0, 0);
equivalent to the non-static overloaded
cullmann added a comment.
Btw., the 2 nullptr in the disconnect can be left out, or?
+ a comment for the future would be nice in the code
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D16927
To: rjvbb, #ktexteditor, #frameworks, cullmann
Cc: cullmann, kde-fram
cullmann accepted this revision.
cullmann added a comment.
This revision is now accepted and ready to land.
Sounds reasonable.
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D16927
To: rjvbb, #ktexteditor, #frameworks, cullmann
Cc: cullmann, kde-frameworks-devel,
ngraham edited the summary of this revision.
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D16927
To: rjvbb, #ktexteditor, #frameworks
Cc: kde-frameworks-devel, kwrite-devel, michaelh, ngraham, bruns, demsking,
head7, cullmann, kfunk, sars, dhaumann
rjvbb created this revision.
rjvbb added reviewers: KTextEditor, Frameworks.
rjvbb added a project: KTextEditor.
Herald added projects: Kate, Frameworks.
Herald added a subscriber: kwrite-devel.
rjvbb requested review of this revision.
REVISION SUMMARY
`ViewPrivate::contextMenu()` has a surprisi