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

Reply via email to