D23825: Split KSslErrorUiData from KTcpSocket

2019-09-10 Thread Volker Krause
This revision was automatically updated to reflect the committed changes. Closed by commit R241:79ee43be4d3f: Split KSslErrorUiData from KTcpSocket (authored by vkrause). REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D23825?vs=65731&id=65775 REVISION DETAIL htt

D23825: Split KSslErrorUiData from KTcpSocket

2019-09-10 Thread Albert Astals Cid
aacid accepted this revision. aacid added a comment. This revision is now accepted and ready to land. Other than that just commit it i'd say, you're just moving code around so no need to "re-review" code that we already accept as "good" for the moment imho REPOSITORY R241 KIO BRANCH mast

D23825: Split KSslErrorUiData from KTcpSocket

2019-09-10 Thread Albert Astals Cid
aacid added inline comments. INLINE COMMENTS > ksslerroruidata.h:26 > + > +template class QList; > +class KTcpSocket; Minor comment, with all the "noise" around QList in Qt6, would it be maybe more future proof to just #include ehre so we don't end up declaring a class that doesn't exist in

D23825: Split KSslErrorUiData from KTcpSocket

2019-09-10 Thread Volker Krause
vkrause created this revision. vkrause added a reviewer: dfaure. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. vkrause requested review of this revision. REVISION SUMMARY The former we want to keep, the latter we want to deprecate, so this is a necessary