Re: [Patch] Locate: Move counting and limit-checking into visitors.

2005-06-26 Thread Bas van Gompel
ChangeLog-entry remains the same: 2005-06-24 Bas van Gompel <[EMAIL PROTECTED]> * locate/locate.c (visit_count, visit_limit): New functions. (locate): Use visit_limit or visit_count. Simplify main loop. (main) Bail out early when limit is reached. diff --exclude=

Re: [Patch] Locate: Move counting and limit-checking into visitors.

2005-06-24 Thread Bas van Gompel
, when you're back, you say you /still/ prefer to do without : > visit_found, I'll prepare another patch. : : That's what I would prefer, to be honest. So here goes. (You /have/ taken note of my objections?) : (the hotel has an SSH client!) Imagine that. :) Another sugges

[Patch] Locate: Move counting and limit-checking into visitors.

2005-06-15 Thread Bas van Gompel
Hi, Following patch moves limit-checking and counting into visitors. When neither is done, a visitor is added which will increment items_accepted once, then remove itself from the list. Possible ChangeLog-entry: 2005-06-15 Bas van Gompel <[EMAIL PROTECTED]> * locate/locate.c

Re: [Patch] Locate: New option: --and

2005-06-12 Thread Bas van Gompel
Op Sat, 11 Jun 2005 23:40:34 +0100 schreef James Youngman in <[EMAIL PROTECTED]>: [locate -a] : It's a good idea. However, "find" uses "-a" for and, like this : : find path predicate1 -a predicate2 -a predicate3 : : which allows this to make sense as well : :find path predicate1 -a ( p

[Patch] Locate: New option: --and

2005-06-11 Thread Bas van Gompel
Hi, Here is the aforementioned patch, to allow a --and (-a) option to locate. I hope you like it... Suggested ChangeLog-entry: 2005-06-11 Bas van Gompel <[EMAIL PROTECTED]> * NEWS, doc/find.texi, locate/locate.1: Document locate --and (-a) * locate/locate.c: Implement

[Patch] Locate: Allow reading database from stdin.

2005-06-08 Thread Bas van Gompel
Hi, Here is a patch which moves reading from the database and generating the base_name into visitors. Support for reading a database from stdin slipped in somehow... Suggested ChangeLog-entry: 2005-06-08 Bas van Gompel <[EMAIL PROTECTED]> * NEWS, locate/locate.1, doc/fin

[Patch] lib/printquoted.h: Include stdio.h.

2005-06-08 Thread Bas van Gompel
Hi, Trying to compile findutils from CVS, I got a compiler-error. Following patch fixes it. Possible ChangeLog-entry: 2005-06-08 Bas van Gompel <[EMAIL PROTECTED]> * lib/printquoted.h: Include stdio.h. diff --exclude='*~' -Ndrup curr/findutils/lib/printquoted.h min

Re: [Patch] Locate: Fold case once at most.

2005-06-06 Thread Bas van Gompel
Op Mon, 6 Jun 2005 06:03:10 +0100 schreef James Youngman in <[EMAIL PROTECTED]>: : On Mon, Jun 06, 2005 at 05:00:15AM +0200, Bas van Gompel wrote: [...] : > : * locate/locate.c: Fold case once, only when needed. : > : > Could you please confirm receiving the above mail, and t

Re: [Patch] Locate: Fold case once at most.

2005-06-05 Thread Bas van Gompel
Hallo James, Op Tue, 31 May 2005 03:44:02 +0200 (MET DST) schreef ik in <[EMAIL PROTECTED]>: [...] : Suggested ChangeLog entry: : : 2005-05-31 Bas van Gompel <[EMAIL PROTECTED]> : : * locate/locate.c: Fold case once, only when needed. Could you please confirm receiving th

[Patch] Locate: Fold case once at most. (was: Re: [Patch] Locate: Read each database only once.)

2005-05-30 Thread Bas van Gompel
e to me. (Except maybe the {munged,original}_filename might be entered into a structure, providing space for more parameters. (This should improve speed for fastcall-compiled code)) Now for the casefolding-patch: Suggested ChangeLog entry: 2005-05-30 Bas van Gompel <[EMAIL PROTECTE

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

2005-05-30 Thread Bas van Gompel
Op Mon, 30 May 2005 09:31:34 +0100 schreef James Youngman in <[EMAIL PROTECTED]>: : 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, cou

[Patch] Locate: Read each database only once.

2005-05-29 Thread Bas van Gompel
intended to be applied. A possible ChangeLog-entry: 2005-05-29 Bas van Gompel <[EMAIL PROTECTED]> * doc/find.texi: Add documentation for locate --print (-p), and the use of non-options with --statistics. * locate/locate.1: ditto. * locate/locate.c: Rea

Re: [Patch] mode_* when compiling findutils with fresh gnulib

2005-05-23 Thread Bas van Gompel
Op Tue, 24 May 2005 03:49:56 +0200 (MET DST) schreef ik in <[EMAIL PROTECTED]>: Disregard, please. I composed this message before receiving <[EMAIL PROTECTED]>. L8r, Buzz. -- ) | | ---/ ---/ Yes, this | This message consists of true | I do not -- | | // really is | and false bi

Re: [Patch] mode_* when compiling findutils with fresh gnulib

2005-05-23 Thread Bas van Gompel
Op Mon, 23 May 2005 01:12:58 -0700 schreef Paul Eggert in <[EMAIL PROTECTED]>: : [EMAIL PROTECTED] (James Youngman) writes: : : > Gnulib guys, what's the significance of the changes made to : > modechange.c? How should callers who previously used MODE_MASK_PLUS : > modify their code to get the sa

Re: [Patch] mode_* when compiling findutils with fresh gnulib

2005-05-22 Thread Bas van Gompel
Op Sun, 22 May 2005 16:57:14 +0200 schreef ik in <[EMAIL PROTECTED]>: [...] : ChangeLog-entry: : : 22-05-2005 Bas van Gompel <[EMAIL PROTECTED]> : : find/parser.c (parse_perm): Accomodate gnulib modifications to : mode_compile and mode_adjust, and the removal of mode

[Patch] mode_* when compiling findutils with fresh gnulib (was: Re: merge from coreutils to gnulib (fd-safer, modechange, readutmp, ...))

2005-05-22 Thread Bas van Gompel
K_PLUS-flag? Is it OK to set the umask-parameter to mode_adjust to zero? ChangeLog-entry: 22-05-2005 Bas van Gompel <[EMAIL PROTECTED]> find/parser.c (parse_perm): Accomodate gnulib modifications to mode_compile and mode_adjust, and the removal of mode_free. diff

Re: [Patch] locate --non-existing (-E): New option.

2005-04-19 Thread Bas van Gompel
Op Mon, 18 Apr 2005 10:05:02 +0100 schreef James Youngman in <[EMAIL PROTECTED]>: : On Mon, Apr 18, 2005 at 04:44:50AM +0200, Bas van Gompel wrote: [patch which implements a ``--non-existing'' (``-E'') option] : I've applied your patch. Mnay thanks for you

Re: [Patch] locate --non-existing (-E): New option.

2005-04-17 Thread Bas van Gompel
Along the way I renamed visit_exists_[no]follow to : visit_existing_[no]follow for consistency, and fixed a a few typos in : the description of --nofollow (``treat broken symbolic links /count/ : as if they were exi/s/ting''). : A possible ChangeLog-entry: : 2005-03-08 Bas van Gompel <[

import-gnulib.sh no longer creates gnulib/m4/Makefile.am

2005-03-07 Thread Bas van Gompel
te as much.) A ChangeLog-entry: 2005-03-08 Bas van Gompel <[EMAIL PROTECTED]> *import-gnulib.sh: Re-enable creation of gnulib/m4/Makefile.am. diff -Ndrupb curr/findutils/import-gnulib.sh mine2/findutils/import-gnulib.sh --- findutils/import-gnulib.sh 2005-02-28 22:56:48.

[Patch] locate --non-existing (-E): New option.

2005-03-07 Thread Bas van Gompel
a few typos in the description of --nofollow (``treat broken symbolic links /count/ as if they were exi/s/ting''). A possible ChangeLog-entry: 2005-03-08 Bas van Gompel <[EMAIL PROTECTED]> *doc/find.texi: Document locate --non-existing (-E). *locate/locate.1: