Re: libfetch ftp patch for less latency

From: Peter Jeremy <peterjeremy_at_optushome.com.au>
Date: Fri, 30 Mar 2007 05:16:13 +1000
On 2007-Mar-29 20:30:26 +0200, Dag-Erling Smrgrav <des_at_des.no> wrote:
>Libfetch used to do what Nate suggests, and it was changed to the
>current behaviour because we encountered servers in the field with
>which it didn't work.

At first glance, the current behaviour does seem unnecessary and
Nate's patch seems an "obvious" improvement.  Having a comment near
_ftp_cwd() explaining the current behaviour and why it isn't possible
to pass a pathname to CWD would have saved Nate some effort and
removed the need for this thread.

Sometimes, it is as important to document why an alternative algorithm
was not chosen as it is to document what the code is doing.

-- 
Peter Jeremy

Received on Thu Mar 29 2007 - 17:16:22 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:39:07 UTC