cfeck added a comment.
Sorry for the late response, but we currently have no maintainer for the FTP code in KIO. From quickly checking, the intention looks good, but if the ERR_* codes are part of an enum, maybe add the special value -1 with a name there, instead of hardcoding the -1 value? It might even fail with compilers that strictly treat enum's as a separate data type. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D9672 To: wcpan, dfaure Cc: cfeck, #frameworks, michaelh, ngraham
