D23579: port ftp slave to new error reporting system

David Faure noreply at phabricator.kde.org
Sat Oct 12 22:39:22 BST 2019


dfaure accepted this revision.
dfaure added a comment.
This revision is now accepted and ready to land.


  Excellent work!

INLINE COMMENTS

> ftp.cpp:2635
> +
> +#pragma message "was this useful? I have no clue where or how keepalive would work with ftp"
> +        //        authenticator->setOption(QStringLiteral("keepalive"), info.keepPassword);

yeah, looks like wishful thinking.... remove?

> sitter wrote in ftp.h:70
> It's a good idea. Would that work though?
> 
> Currently the results rely on the implicit move operator when collecting the returned result
> 
>   result = ftpGet()
> 
> if the members are const we couldn't move/copy like that anymore.
> 
> If we consider the mutability a problem I think I'd just make the members private and give them getters. I don't mind much either way.

Copying bool+int+QString seems rather cheap, we lived with that until C++11 move semantics ;-)

Private members and inline getters sounds good to me.

But yeah, no big deal, we can also trust the programmer :-)

REPOSITORY
  R241 KIO

BRANCH
  ftp

REVISION DETAIL
  https://phabricator.kde.org/D23579

To: sitter, dfaure
Cc: anthonyfieroni, dfaure, 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/d8e57375/attachment.html>


More information about the Kde-frameworks-devel mailing list