Re: new module timer-time

[Bruno Haible]

Pádraig Brady wrote:
> [timer-time.diff attachment]

> +Check for timer_settime.  Within HAVE_TIMER_SETTIME one can
> +assume timer_create is available too, as that is a
> +prerequisite to use timer_settime.

And timer_gettime as well. I've verified in the platformsxsymbols matrix
that timer_create, timer_gettime, timer_settime are available in the same
way on all platforms:

libc                aix-4.3.2
libc                aix-5.1.0
libc                aix-5.2.0
libc                aix-5.3.0a
libc                aix-6.1.0
libc                aix-7.1.0
libc                cygwin-1.5
libc                cygwin-1.7
librt               glibc-2.3.6
libc                hpux-10.20
libc                hpux-11.00
librt               hpux-11.00
libc                hpux-11.11
librt               hpux-11.11
libc                hpux-11.23
librt               hpux-11.23
libc                hpux-11.31
librt               hpux-11.31
libc                irix-6.5
libc                netbsd-3.0
libc                netbsd-5.0
libc                netbsd-5.0.1
libc                openbsd-3.8
librt               osf1-4.0d
librt               osf1-5.1a
libposix4           solaris-2.10
librt               solaris-2.10
libc                solaris-2.11_2010_11
libposix4           solaris-2.11_2010_11
librt               solaris-2.11_2010_11
libposix4           solaris-2.5.1
libposix4           solaris-2.6
libposix4           solaris-2.7
librt               solaris-2.7
libposix4           solaris-2.8
librt               solaris-2.8
libposix4           solaris-2.9
librt               solaris-2.9
MISSING in          beos freebsd-5.2.1 freebsd-6.0 freebsd-6.4 interix-3.5 
irix-5.3 macosx-10.3 macosx-10.4 macosx-10.5 mingw nsk-G06 pips solaris-2.4

> +# For a program named, say foo, you should add a line like the following
> +# in the corresponding Makefile.am file:
> +# foo_LDADD = $(LDADD) $(LIB_TIMER_TIME)

This comment is redundant: The 'Link' field in the module specification
also says it.

> +Include:
> +

This looks wrong: Users of these functions must include <time.h>. So that's
what you need to specify here.

While we are at it: What can <time.h> do to define these functions on platforms
that lack it?

Bruno
-- 
In memoriam Paul Schneider 
<http://en.wikipedia.org/wiki/Paul_Schneider_(pastor)>