apol requested changes to this revision. apol added inline comments. This revision now requires changes to proceed.
INLINE COMMENTS > kurlrequester.cpp:132 > > - connect(sender, SIGNAL(returnPressed()), > - receiver, SIGNAL(returnPressed())); > - connect(sender, SIGNAL(returnPressed(QString)), > - receiver, SIGNAL(returnPressed(QString))); > + connect(combo, static_cast<void > (KComboBox::*)()>(&KComboBox::returnPressed), > + receiver, static_cast<void > (KUrlRequester::*)()>(&KUrlRequester::returnPressed)); We can use qOverload now (we depend on Qt 5.7). It will end being much more readable. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D7968 To: mwolff, apol Cc: apol, #frameworks