vkrause added a comment.
In D24873#552697 <https://phabricator.kde.org/D24873#552697>, @ahmadsamir wrote: > In D24873#552410 <https://phabricator.kde.org/D24873#552410>, @vkrause wrote: > > > This will compile, but it will only work correctly once TCPSlaveBase has been ported to QSslSocket too. > > > If you say so. Well, socket() returns the KTcpSocket that TCPSlaveBase created, so this cast will always return a nullptr. > Anyway, I've started porting TcpSlaveBase. :) Awesome! There seems to be only one other place that also assumes socket() returns a KTcpSocket (https://lxr.kde.org/source/kde/pim/kdepim-runtime/kioslave/pop3/pop3.cpp#0625). REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D24873 To: ahmadsamir, dfaure, vkrause Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
