Date: Thu, 29 Mar 2007 21:43:45 +0200 From: des@des.no (Dag-Erling =?iso-8859-1?Q?Sm=F8rgrav?=) To: Peter Jeremy <peterjeremy@optushome.com.au> Cc: current@freebsd.org, Nate Lawson <nate@root.org> Subject: Re: libfetch ftp patch for less latency Message-ID: <86y7lfomxa.fsf@dwp.des.no> In-Reply-To: <20070329191613.GB827@turion.vk2pj.dyndns.org> (Peter Jeremy's message of "Fri, 30 Mar 2007 05:16:13 %2B1000") References: <460AE39B.4070706@root.org> <86odmcqylx.fsf@dwp.des.no> <200703291905.00192.pieter@degoeje.nl> <86k5wzq4vx.fsf@dwp.des.no> <20070329191613.GB827@turion.vk2pj.dyndns.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Peter Jeremy <peterjeremy@optushome.com.au> writes: > 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. Isn't that why we have CVS logs? ---------------------------- revision 1.92 date: 2005/08/12 12:48:50; author: des; state: Exp; lines: +152 -29 Change directory one level at a time, and use CDUP to back out. This is a work in progress; it partially fixed bin/83278 and is a prerequisite to fixing bin/83277. PR: bin/83277, bin/83278 ---------------------------- DES --=20 Dag-Erling Sm=F8rgrav - des@des.no
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?86y7lfomxa.fsf>