On 17 February 2012 15:15, Daniel Stenberg <daniel_at_haxx.se> wrote:
> Hi friends,
>
> As I promised earlier, here is my take at doing the same fix to sftp_write
> as we've worked on for sftp_read.
>
> I'll appreciate eyeballs and testing!
It looks OK to me and passes my test suite.
One thing: why do we need that final while-loop? Are we keeping that
around (as with sftp_read) solely for the case where nothing was
acked? When would that happen? Can chunk->len ever be zero? Is this
worth keeping the loop around for; unlike sftp_read, we can return 0
without it signalling completion?
Also, why do we increment acked? I can't see any execution of the loop
that could have acked as anything other than zero at the point it is
incremented.
Alex
-- Swish - Easy SFTP for Windows Explorer (http://www.swish-sftp.org) _______________________________________________ libssh2-devel http://cool.haxx.se/cgi-bin/mailman/listinfo/libssh2-develReceived on 2012-02-19