[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Bug-gnulib] 03-getndelim2-c89.diff
From: |
Bruno Haible |
Subject: |
Re: [Bug-gnulib] 03-getndelim2-c89.diff |
Date: |
Thu, 24 Jul 2003 14:39:33 +0200 |
User-agent: |
KMail/1.5 |
Derek Robert Price wrote:
> -getloadavg (loadavg, nelem)
> - double loadavg[];
> - int nelem;
> +getloadavg (double *loadavg, int nelem)
I'd prefer "double loadavg[]"
because that's the way the libc documentation writes it, and the array
has a fixed length.
> char *
> -getusershell ()
> +getusershell (void)
There's absolutely no point is changing function _definitions_ with
no arguments to use (void). Both ways to write it are equivalent from
the language point of view, and (void) is just useless clutter for the
human reader. Please drop this.
> __ptr_t
> -__memchr (s, c_in, n)
> - const __ptr_t s;
> - int c_in;
> - size_t n;
> +__memchr (const void *s, int c_in, size_t n)
Why do you replace one instance of __ptr_t when you keep the other one?
Bruno
- Re: [Bug-gnulib] 03-getndelim2-c89.diff, (continued)
Re: [Bug-gnulib] 03-getndelim2-c89.diff, Derek Robert Price, 2003/07/22
Re: [Bug-gnulib] 03-getndelim2-c89.diff, Paul Eggert, 2003/07/22
Re: [Bug-gnulib] 03-getndelim2-c89.diff, Derek Robert Price, 2003/07/22
Re: [Bug-gnulib] 03-getndelim2-c89.diff, Paul Eggert, 2003/07/22
Re: [Bug-gnulib] 03-getndelim2-c89.diff, Derek Robert Price, 2003/07/23
Re: [Bug-gnulib] 03-getndelim2-c89.diff,
Bruno Haible <=
Re: [Bug-gnulib] 03-getndelim2-c89.diff, James Youngman, 2003/07/24
Re: [Bug-gnulib] 03-getndelim2-c89.diff, Bruno Haible, 2003/07/24
Re: [Bug-gnulib] 03-getndelim2-c89.diff, Derek Robert Price, 2003/07/24
Re: [Bug-gnulib] 03-getndelim2-c89.diff, Tyler Berry, 2003/07/24
Re: [Bug-gnulib] 03-getndelim2-c89.diff, Paul Eggert, 2003/07/24
Re: [Bug-gnulib] 03-getndelim2-c89.diff, Derek Robert Price, 2003/07/24
Re: [Bug-gnulib] 03-getndelim2-c89.diff, Derek Robert Price, 2003/07/24
Re: [Bug-gnulib] 03-getndelim2-c89.diff, Paul Eggert, 2003/07/24
Re: [Bug-gnulib] 03-getndelim2-c89.diff, Derek Robert Price, 2003/07/25
Re: [Bug-gnulib] 03-getndelim2-c89.diff, Bruno Haible, 2003/07/22