D24588: [SslUi] Overload askIgnoreSslErrors() to take a QSslSocket
Ahmad Samir
noreply at phabricator.kde.org
Sat Oct 12 23:33:45 BST 2019
ahmadsamir added a comment.
In D24588#546229 <https://phabricator.kde.org/D24588#546229>, @dfaure wrote:
> Hmm, why not just do `KSslErrorUiData uiData(socket);` in the caller, or even just `askIgnoreSslErrors(KSslErrorUiData(socket), storedRules)`.
> Don't know, just wondering (not objecting). @vkrause, any opinion?
My reasoning is, konversation only needs to know about SslUi::askIgnoreSslErrors and not KSslErrorUiData (implementation detail in SslUi). It's more "streamlined" for users of askIgnoreSslErrors to pass it an SSL socket rather than a KSslErrorUiData object. Anyway, I don't mind either way.
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D24588
To: ahmadsamir, dfaure, vkrause
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191012/cea20212/attachment.html>
More information about the Kde-frameworks-devel
mailing list