Replaced by D24887 which instead creates a more generic system to verify slave behavior.

remove warning

I've actually started work on a more general system for this. Will update soon.

works well enough it seems

When I put the code in a minimal standalone program I can confirm that the code works just fine. Are you sure your testing methodology is sound @ngraham ?

Thanks!

Do we have a request for this outside kbibtex? My attitude towards adding things to ECM is always "is there more than one user".

In D24626#546854, @bcooksley wrote:

Thanks for this - i'll get this integrated soon.

Please note that Ruby is totally unavailable on our Windows builders - will you be making the presence of this gem optional (just disabling the unit tests when it is unavailable) in KIO?

move away from helper functions to redirect to q-> and instaed call via q-> directly

• rebase
• ftpSendCmd now returns a bool (continues to require_result)
• openConnection now emits opened()
• listDir now finalize()s
• debug--
• ftpSendCmd's lazy login now checks the return value of the recrusive cmd call

@nicolasfella do you have thoughts?

My point is that your lamentations have nothing to do with this class but with Q*Animation on your system. So you need to find out what's wrong and talk to Qt. I am 100% against a workaround that degrades the user experience when the bug isn't even in this class.

^ The diff for people not fluent in reverse-markdown ;)

The demo doesn't even use this widget

❤

I meant the svg from breeze-icons. The one in the repo would be from kde4 or even 3 times.

Feel free to update the pngs. I haven't looked but I'm certain the PNGs are ancient icons from before breeze, so go with what breeze has.

Yeah, we used Jonathan's test instance basically like a mailing list. It works very well.

Is this done then from our side?

looks good