strcasestr [was: strstr speedup]

2008-01-14 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Bruno Haible on 1/11/2008 4:21 AM: | Eric Blake wrote: |> | should I go ahead and prepare a patch for strcasestr, c-strcasestr, and |> | memcasecmp to utilize str-two-way.h? | | Yes for strcasestr and c-strcasestr. Here's what I'm commit

Re: strsignal module

2008-01-14 Thread Bruno Haible
Colin Watson wrote: > I think the attached patch is right (I tested it with LIBS=-lstrsignal > after installing a bogus /usr/lib/libstrsignal.a with a strsignal > function that always returns NULL), but would appreciate review from > those more fluent in Autoconf than I am. Thanks for the fast wor

Re: strsignal module

2008-01-14 Thread Colin Watson
On Mon, Jan 14, 2008 at 11:45:17PM +0100, Bruno Haible wrote: > The mention of a file "strsignal.h" is wrong; I'm fixing it like this: A vestige of an earlier version of the module. Thanks for spotting it. -- Colin Watson [EMAIL PROTECTED]

Re: strsignal module

2008-01-14 Thread Colin Watson
On Tue, Jan 15, 2008 at 12:28:55AM +0100, Bruno Haible wrote: > You fixed the first portability problem: the function is missing on some > platforms. But there is another one. > > I want to use the replacement in GNU clisp, so I did "man strsignal", and it > says: > >"On some systems (but

Re: strsignal module

2008-01-14 Thread Bruno Haible
Colin Watson wrote: > I use strsignal in man-db, and would like a Gnulib module to cope with > its portability problems. You fixed the first portability problem: the function is missing on some platforms. But there is another one. I want to use the replacement in GNU clisp, so I did "man strsigna

Re: argp_error and argp_failure xgettext options

2008-01-14 Thread Colin Watson
On Mon, Jan 14, 2008 at 11:37:03PM +0100, Bruno Haible wrote: > Hi Colin, > > It looks like argp_error and argp_failure should declare themselves as > > requiring the c-format flag. This patch implements that. > > > > 2008-01-14 Colin Watson <[EMAIL PROTECTED]> > > > > * modules/argp: Add a

Re: strsignal module

2008-01-14 Thread Bruno Haible
> I've attached an updated patch. The mention of a file "strsignal.h" is wrong; I'm fixing it like this: 2008-01-14 Bruno Haible <[EMAIL PROTECTED]> * modules/strsignal (Include): Change to . *** modules/strsignal.orig 2008-01-14 23:43:15.0 +0100 --- modules/strsignal

Re: argp_error and argp_failure xgettext options

2008-01-14 Thread Bruno Haible
Hi Colin, > It looks like argp_error and argp_failure should declare themselves as > requiring the c-format flag. This patch implements that. > > 2008-01-14 Colin Watson <[EMAIL PROTECTED]> > > * modules/argp: Add a notice recommending to change XGETTEXT_OPTIONS. > Invoke AM_GETTE

Re: getdate.y fails to compile on QNX 6

2008-01-14 Thread Jim Meyering
Paul Eggert <[EMAIL PROTECTED]> wrote: > Matt Kraai <[EMAIL PROTECTED]> writes: >> , which is still present in the latest Gnulib sources. time_t is >> equivalent to an unsigned long, which causes both of the comparisons >> to fail. > > It looks like someone has to port getdate.y to QNX 6, then. D

argp_error and argp_failure xgettext options

2008-01-14 Thread Colin Watson
Hi, It looks like argp_error and argp_failure should declare themselves as requiring the c-format flag. This patch implements that. 2008-01-14 Colin Watson <[EMAIL PROTECTED]> * modules/argp: Add a notice recommending to change XGETTEXT_OPTIONS. Invoke AM_GETTEXT_OPTION if it e