Re: libfetch ftp patch for less latency



Peter Jeremy <peterjeremy@xxxxxxxxxxxxxxxx> 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
--
Dag-Erling Smørgrav - des@xxxxxx
_______________________________________________
freebsd-current@xxxxxxxxxxx mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscribe@xxxxxxxxxxx"



Relevant Pages

  • Re: ATWT --- CASTING UPDATE
    ... how many DOOL fans who no longer watch the ... show will tune in to see Patch and Kayla? ... I can't imagine that they would do so. ... mean any storyline bringing Patch back, undead, explaining what happened ...
    (rec.arts.tv.soaps.cbs)
  • Re: select timings
    ... On Mon, 9 Jul 2007, Peter Jeremy wrote: ... a selecton each descriptor. ... Well this test is useful because it shows the expected worst case for this patch which is very encouraging since one version of it is not slower than the original select code. ... Contrasted with the original code which calls poll all over again on each. ...
    (freebsd-arch)
  • [PATCH] [1/3] x86: Improve CONFIG_KEXEC_JUMP description
    ... I've been puzzling what CONFIG_KEXEC_JUMP was good for. ... The Kconfig help was not very helpful in explaining why one ... This patch improves ...
    (Linux-Kernel)
  • Re: Reminder: non-mpsafetty drivers to be connected on Sunday
    ... In message, Peter Jeremy writes ... patch I'm going to commit. ... Comments on the patch are very welcome! ... This patch just disconnects the majority of the serial drivers from ...
    (freebsd-arch)
  • Re: Reminder: non-mpsafetty drivers to be connected on Sunday
    ... In message, Peter Jeremy writes ... patch I'm going to commit. ... Comments on the patch are very welcome! ... This patch just disconnects the majority of the serial drivers from ...
    (freebsd-current)