bug-findutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Patch] Locate: Read each database only once.


From: Bas van Gompel
Subject: Re: [Patch] Locate: Read each database only once.
Date: Mon, 30 May 2005 21:11:44 +0200 (MET DST)
User-agent: slrn/0.9.8.1 (Win32) Hamster/2.0.6.0 KorrNews/4.2

Op Mon, 30 May 2005 09:31:34 +0100 schreef James Youngman
in <address@hidden>:
:  On Sun, May 29, 2005 at 11:11:52PM +0200, Bas van Gompel wrote:
[read each db once]

:  A well-constructed patch, many thanks!  I have applied it.  In the
:  future, when making a user-visible change, could you add a comment in
:  the NEWS file describing the impact of the change?    Thanks.

Thank you! I will try to remember to do that...

:  Lastly, do you think there is scope for removing process_simple()
:  entirely?  Special-casing argc <= 1 by having a dedicated function
:  seems a little inelegant.

process_simple() should be /slightly/ faster...


L8r,

Buzz.
-- 
  ) |  | ---/ ---/  Yes, this | This message consists of true | I do not
--  |  |   /    /   really is |   and false bits entirely.    | mail for
  ) |  |  /    /    a 72 by 4 +-------------------------------+ any1 but
--  \--| /--- /---  .sigfile. |   |perl -pe "s.u(z)\1.as."    | me. 4^re




reply via email to

[Prev in Thread] Current Thread [Next in Thread]