[PATCH 00/20] More prototype fixes

[PATCH 00/20] More prototype fixes

[Rasmus Villemoes]

The previously mentioned perl script can also be applied to the man3
pages. This round of patches comes mostly from applying it to
man3/[a-g].*, but I've also included other stuff I've stumbled across.

The perl script itself follows soon.

Rasmus Villemoes (20):
  open.2: spfix
  set_mempolicy.2: Fix prototype for set_mempolicy(2)
  remap_file_pages.2: Fix prototype
  swapon.2: remove header from synopsis
  a64l.3: fix prototype for a64l()
  argz_*: Fix prototypes
  adjtime.3: add required header
  bstring.3: Fix prototypes
  dl_iterate_phdr.3: tfix
  inet_net_pton.3: srcfix, cfix
  envz_add.3: fix prototypes
  fpathconf.3: fix prototype
  fseek.3: fix prototype
  gcvt.3: fix prototype
  getaddrinfo_a.3: fix prototype
  getnameinfo.3: fix prototype
  getrpcent.3: fix prototype
  getspnam.3: fix prototype
  getutent.3: fix prototypes
  getrpcport.3: add #include and fix prototype

 man2/open.2             | 2 +-
 man2/remap_file_pages.2 | 2 +-
 man2/set_mempolicy.2    | 4 ++--
 man2/swapon.2           | 2 --
 man3/a64l.3             | 2 +-
 man3/adjtime.3          | 2 ++
 man3/argz_add.3         | 6 +++---
 man3/bstring.3          | 6 +++---
 man3/dl_iterate_phdr.3  | 4 ++--
 man3/envz_add.3         | 4 ++--
 man3/fpathconf.3        | 2 +-
 man3/fseek.3            | 2 +-
 man3/gcvt.3             | 2 +-
 man3/getaddrinfo_a.3    | 4 ++--
 man3/getnameinfo.3      | 4 ++--
 man3/getrpcent.3        | 2 +-
 man3/getrpcport.3       | 6 ++++--
 man3/getspnam.3         | 2 +-
 man3/getutent.3         | 6 +++---
 man3/inet_net_pton.3    | 5 ++---
 20 files changed, 35 insertions(+), 34 deletions(-)

-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 01/20] open.2: spfix

[Rasmus Villemoes]

---
 man2/open.2 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man2/open.2 b/man2/open.2
index 35061c3..d22bb1b 100644
--- a/man2/open.2
+++ b/man2/open.2
@@ -717,7 +717,7 @@ as described above).
 requires support by the underlying filesystem;
 only a subset of Linux filesystems provide that support.
 In the initial implementation, support was provided in
-the ex2, ext3, ext4, UDF, Minix, and shmem filesystems.
+the ext2, ext3, ext4, UDF, Minix, and shmem filesystems.
 XFS support was added
 .\" commit 99b6436bc29e4f10e4388c27a3e4810191cc4788
 .\" commit ab29743117f9f4c22ac44c13c1647fb24fb2bafe
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 01/20] open.2: spfix

[Michael Kerrisk (man-pages)]

By chance, I just caught this one myself yesterday!

Thanks,

Michael


On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> ---
>  man2/open.2 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man2/open.2 b/man2/open.2
> index 35061c3..d22bb1b 100644
> --- a/man2/open.2
> +++ b/man2/open.2
> @@ -717,7 +717,7 @@ as described above).
>  requires support by the underlying filesystem;
>  only a subset of Linux filesystems provide that support.
>  In the initial implementation, support was provided in
> -the ex2, ext3, ext4, UDF, Minix, and shmem filesystems.
> +the ext2, ext3, ext4, UDF, Minix, and shmem filesystems.
>  XFS support was added
>  .\" commit 99b6436bc29e4f10e4388c27a3e4810191cc4788
>  .\" commit ab29743117f9f4c22ac44c13c1647fb24fb2bafe
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 02/20] set_mempolicy.2: Fix prototype for set_mempolicy(2)

[Rasmus Villemoes]

The nodemask argument is const. The return type in numaif.h is long.
---
 man2/set_mempolicy.2 | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/man2/set_mempolicy.2 b/man2/set_mempolicy.2
index 843208c..b17cf30 100644
--- a/man2/set_mempolicy.2
+++ b/man2/set_mempolicy.2
@@ -31,8 +31,8 @@ set_mempolicy \- set default NUMA memory policy for a process and its children
 .nf
 .B "#include <numaif.h>"
 .sp
-.BI "int set_mempolicy(int " mode ", unsigned long *" nodemask ,
-.BI "                  unsigned long " maxnode );
+.BI "long set_mempolicy(int " mode ", const unsigned long *" nodemask ,
+.BI "                   unsigned long " maxnode );
 .sp
 Link with \fI\-lnuma\fP.
 .fi
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 02/20] set_mempolicy.2: Fix prototype for set_mempolicy(2)

[Michael Kerrisk (man-pages)]

Thanks! Applied.

Cheers,

Michael


On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The nodemask argument is const. The return type in numaif.h is long.
> ---
>  man2/set_mempolicy.2 | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/man2/set_mempolicy.2 b/man2/set_mempolicy.2
> index 843208c..b17cf30 100644
> --- a/man2/set_mempolicy.2
> +++ b/man2/set_mempolicy.2
> @@ -31,8 +31,8 @@ set_mempolicy \- set default NUMA memory policy for a process and its children
>  .nf
>  .B "#include <numaif.h>"
>  .sp
> -.BI "int set_mempolicy(int " mode ", unsigned long *" nodemask ,
> -.BI "                  unsigned long " maxnode );
> +.BI "long set_mempolicy(int " mode ", const unsigned long *" nodemask ,
> +.BI "                   unsigned long " maxnode );
>  .sp
>  Link with \fI\-lnuma\fP.
>  .fi
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 03/20] remap_file_pages.2: Fix prototype

[Rasmus Villemoes]

The pgoff argument has type size_t, not ssize_t (and in the kernel it
is unsigned long).
---
 man2/remap_file_pages.2 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man2/remap_file_pages.2 b/man2/remap_file_pages.2
index 50c6ce2..e7879fd 100644
--- a/man2/remap_file_pages.2
+++ b/man2/remap_file_pages.2
@@ -34,7 +34,7 @@ remap_file_pages \- create a nonlinear file mapping
 .B #include <sys/mman.h>
 .sp
 .BI "int remap_file_pages(void *" addr ", size_t " size ", int " prot ,
-.BI "                     ssize_t " pgoff ", int " flags );
+.BI "                     size_t " pgoff ", int " flags );
 .fi
 .SH DESCRIPTION
 The
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 03/20] remap_file_pages.2: Fix prototype

[Michael Kerrisk (man-pages)]

Thanks, Rasmus! Applied.

Cheers,

Michael


On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The pgoff argument has type size_t, not ssize_t (and in the kernel it
> is unsigned long).
> ---
>  man2/remap_file_pages.2 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man2/remap_file_pages.2 b/man2/remap_file_pages.2
> index 50c6ce2..e7879fd 100644
> --- a/man2/remap_file_pages.2
> +++ b/man2/remap_file_pages.2
> @@ -34,7 +34,7 @@ remap_file_pages \- create a nonlinear file mapping
>  .B #include <sys/mman.h>
>  .sp
>  .BI "int remap_file_pages(void *" addr ", size_t " size ", int " prot ,
> -.BI "                     ssize_t " pgoff ", int " flags );
> +.BI "                     size_t " pgoff ", int " flags );
>  .fi
>  .SH DESCRIPTION
>  The
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 04/20] swapon.2: remove header from synopsis

[Rasmus Villemoes]

The <asm/page.h> header is not readily available, and the comment
seems to indicate that it is for getting PAGE_SIZE. But it is never
mentioned why one would need to know that, and it is in any case
better obtained using sysconf(), provided by <unistd.h>.
---
 man2/swapon.2 | 2 --
 1 file changed, 2 deletions(-)

diff --git a/man2/swapon.2 b/man2/swapon.2
index 9ab380c..18f8655 100644
--- a/man2/swapon.2
+++ b/man2/swapon.2
@@ -40,8 +40,6 @@ swapon, swapoff \- start/stop swapping to file/device
 .SH SYNOPSIS
 .B #include <unistd.h>
 .br
-.B #include <asm/page.h>     /* to find PAGE_SIZE */
-.br
 .B #include <sys/swap.h>
 .sp
 .BI "int swapon(const char *" path ", int " swapflags );
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 04/20] swapon.2: remove header from synopsis

[Michael Kerrisk (man-pages)]

On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The <asm/page.h> header is not readily available, and the comment
> seems to indicate that it is for getting PAGE_SIZE. But it is never
> mentioned why one would need to know that, and it is in any case
> better obtained using sysconf(), provided by <unistd.h>.

Hi Rasmus,

Agreed. I'm not sure how that strangeness ever got into the page.
Patch applied. Thanks!

Cheers,

Michael



> ---
>  man2/swapon.2 | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/man2/swapon.2 b/man2/swapon.2
> index 9ab380c..18f8655 100644
> --- a/man2/swapon.2
> +++ b/man2/swapon.2
> @@ -40,8 +40,6 @@ swapon, swapoff \- start/stop swapping to file/device
>  .SH SYNOPSIS
>  .B #include <unistd.h>
>  .br
> -.B #include <asm/page.h>     /* to find PAGE_SIZE */
> -.br
>  .B #include <sys/swap.h>
>  .sp
>  .BI "int swapon(const char *" path ", int " swapflags );
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 05/20] a64l.3: fix prototype for a64l()

[Rasmus Villemoes]

The argument is const, both according to POSIX and the glibc headers.
---
 man3/a64l.3 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man3/a64l.3 b/man3/a64l.3
index 61a1872..0a21f82 100644
--- a/man3/a64l.3
+++ b/man3/a64l.3
@@ -12,7 +12,7 @@ a64l, l64a \- convert between long and base-64
 .SH SYNOPSIS
 .B #include <stdlib.h>
 .sp
-.BI "long a64l(char *" str64 );
+.BI "long a64l(const char *" str64 );
 .sp
 .BI "char *l64a(long " value );
 .sp
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 05/20] a64l.3: fix prototype for a64l()

[Michael Kerrisk (man-pages)]

Thanks, Rasmus! Applied.

Cheers,

Michael

On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The argument is const, both according to POSIX and the glibc headers.
> ---
>  man3/a64l.3 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man3/a64l.3 b/man3/a64l.3
> index 61a1872..0a21f82 100644
> --- a/man3/a64l.3
> +++ b/man3/a64l.3
> @@ -12,7 +12,7 @@ a64l, l64a \- convert between long and base-64
>  .SH SYNOPSIS
>  .B #include <stdlib.h>
>  .sp
> -.BI "long a64l(char *" str64 );
> +.BI "long a64l(const char *" str64 );
>  .sp
>  .BI "char *l64a(long " value );
>  .sp
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 06/20] argz_*: Fix prototypes

[Rasmus Villemoes]

Update the prototypes of argz_{delete,extract,next} to agree with
glibc headers and manual.
---
 man3/argz_add.3 | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/man3/argz_add.3 b/man3/argz_add.3
index 3d07fc0..a28f9e2 100644
--- a/man3/argz_add.3
+++ b/man3/argz_add.3
@@ -33,14 +33,14 @@ argz_next, argz_replace, argz_stringify \- functions to handle an argz list
 .BI "error_t argz_create_sep(const char *" str ", int " sep ", char **" argz ,
 .BI "                     size_t *" argz_len );
 .sp
-.BI "error_t argz_delete(char **" argz ", size_t *" argz_len ", char *" entry );
+.BI "void argz_delete(char **" argz ", size_t *" argz_len ", char *" entry );
 .sp
-.BI "void argz_extract(char *" argz ", size_t " argz_len ", char  **" argv );
+.BI "void argz_extract(const char *" argz ", size_t " argz_len ", char  **" argv );
 .sp
 .BI "error_t argz_insert(char **" argz ", size_t *" argz_len ", char *" before ,
 .BI "                     const char *" entry );
 .sp
-.BI "char *argz_next(char *" argz ", size_t " argz_len ", const char *" entry );
+.BI "char *argz_next(const char *" argz ", size_t " argz_len ", const char *" entry );
 .sp
 .BI "error_t argz_replace(char **" argz ", size_t *" argz_len \
 ", const char *" str ,
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 06/20] argz_*: Fix prototypes

[Michael Kerrisk (man-pages)]

Thanks. Applied.

Cheers,

Michael

On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> Update the prototypes of argz_{delete,extract,next} to agree with
> glibc headers and manual.
> ---
>  man3/argz_add.3 | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/man3/argz_add.3 b/man3/argz_add.3
> index 3d07fc0..a28f9e2 100644
> --- a/man3/argz_add.3
> +++ b/man3/argz_add.3
> @@ -33,14 +33,14 @@ argz_next, argz_replace, argz_stringify \- functions to handle an argz list
>  .BI "error_t argz_create_sep(const char *" str ", int " sep ", char **" argz ,
>  .BI "                     size_t *" argz_len );
>  .sp
> -.BI "error_t argz_delete(char **" argz ", size_t *" argz_len ", char *" entry );
> +.BI "void argz_delete(char **" argz ", size_t *" argz_len ", char *" entry );
>  .sp
> -.BI "void argz_extract(char *" argz ", size_t " argz_len ", char  **" argv );
> +.BI "void argz_extract(const char *" argz ", size_t " argz_len ", char  **" argv );
>  .sp
>  .BI "error_t argz_insert(char **" argz ", size_t *" argz_len ", char *" before ,
>  .BI "                     const char *" entry );
>  .sp
> -.BI "char *argz_next(char *" argz ", size_t " argz_len ", const char *" entry );
> +.BI "char *argz_next(const char *" argz ", size_t " argz_len ", const char *" entry );
>  .sp
>  .BI "error_t argz_replace(char **" argz ", size_t *" argz_len \
>  ", const char *" str ,
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 07/20] adjtime.3: add required header

[Rasmus Villemoes]

The prototype for adjtime(3) is declared in <sys/time.h>.
---
 man3/adjtime.3 | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/man3/adjtime.3 b/man3/adjtime.3
index 878955b..aa2311f 100644
--- a/man3/adjtime.3
+++ b/man3/adjtime.3
@@ -27,6 +27,8 @@
 adjtime \- correct the time to synchronize the system clock
 .SH SYNOPSIS
 .nf
+.B #include <sys/time.h>
+.sp
 .BI "int adjtime(const struct timeval *" delta ", struct timeval *" olddelta );
 .fi
 .sp
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 07/20] adjtime.3: add required header

[Michael Kerrisk (man-pages)]

Thanks, Rasmus! Applied.

Cheers,

Michael


On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The prototype for adjtime(3) is declared in <sys/time.h>.
> ---
>  man3/adjtime.3 | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/man3/adjtime.3 b/man3/adjtime.3
> index 878955b..aa2311f 100644
> --- a/man3/adjtime.3
> +++ b/man3/adjtime.3
> @@ -27,6 +27,8 @@
>  adjtime \- correct the time to synchronize the system clock
>  .SH SYNOPSIS
>  .nf
> +.B #include <sys/time.h>
> +.sp
>  .BI "int adjtime(const struct timeval *" delta ", struct timeval *" olddelta );
>  .fi
>  .sp
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 08/20] bstring.3: Fix prototypes

[Rasmus Villemoes]

The length parameter n has type size_t in bcmp(), bcopy() and bzero().
---
 man3/bstring.3 | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/man3/bstring.3 b/man3/bstring.3
index 451e6a1..70a6535 100644
--- a/man3/bstring.3
+++ b/man3/bstring.3
@@ -37,11 +37,11 @@ memmove, memset \- byte string operations
 .nf
 .B #include <string.h>
 .sp
-.BI "int bcmp(const void *" s1 ", const void *" s2 ", int " n );
+.BI "int bcmp(const void *" s1 ", const void *" s2 ", size_t " n );
 .sp
-.BI "void bcopy(const void *" src ", void *" dest ", int " n );
+.BI "void bcopy(const void *" src ", void *" dest ", size_t " n );
 .sp
-.BI "void bzero(void *" s ", int " n );
+.BI "void bzero(void *" s ", size_t " n );
 .sp
 .BI "void *memccpy(void *" dest ", const void *" src ", int " c ", size_t " n );
 .sp
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 08/20] bstring.3: Fix prototypes

[Michael Kerrisk (man-pages)]

Thanks, Rasmus! Applied.

Cheers,

Michael


On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The length parameter n has type size_t in bcmp(), bcopy() and bzero().
> ---
>  man3/bstring.3 | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/man3/bstring.3 b/man3/bstring.3
> index 451e6a1..70a6535 100644
> --- a/man3/bstring.3
> +++ b/man3/bstring.3
> @@ -37,11 +37,11 @@ memmove, memset \- byte string operations
>  .nf
>  .B #include <string.h>
>  .sp
> -.BI "int bcmp(const void *" s1 ", const void *" s2 ", int " n );
> +.BI "int bcmp(const void *" s1 ", const void *" s2 ", size_t " n );
>  .sp
> -.BI "void bcopy(const void *" src ", void *" dest ", int " n );
> +.BI "void bcopy(const void *" src ", void *" dest ", size_t " n );
>  .sp
> -.BI "void bzero(void *" s ", int " n );
> +.BI "void bzero(void *" s ", size_t " n );
>  .sp
>  .BI "void *memccpy(void *" dest ", const void *" src ", int " c ", size_t " n );
>  .sp
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 09/20] dl_iterate_phdr.3: tfix

[Rasmus Villemoes]

---
 man3/dl_iterate_phdr.3 | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/man3/dl_iterate_phdr.3 b/man3/dl_iterate_phdr.3
index d57bf66..be06d1f 100644
--- a/man3/dl_iterate_phdr.3
+++ b/man3/dl_iterate_phdr.3
@@ -32,8 +32,8 @@ dl_iterate_phdr \- walk through list of shared objects
 
 .BI "int dl_iterate_phdr("
 .BI "          int (*" callback ") (struct dl_phdr_info *" info ,
-.BI "                           size_t " size ", void *" data ,
-.BI "          void *" data ;
+.BI "                           size_t " size ", void *" data "),"
+.BI "          void *" data ");"
 .fi
 .SH DESCRIPTION
 The
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 09/20] dl_iterate_phdr.3: tfix

[Michael Kerrisk (man-pages)]

Thanks, Rasmus! Applied.

Cheers,

Michael


On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> ---
>  man3/dl_iterate_phdr.3 | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/man3/dl_iterate_phdr.3 b/man3/dl_iterate_phdr.3
> index d57bf66..be06d1f 100644
> --- a/man3/dl_iterate_phdr.3
> +++ b/man3/dl_iterate_phdr.3
> @@ -32,8 +32,8 @@ dl_iterate_phdr \- walk through list of shared objects
>  
>  .BI "int dl_iterate_phdr("
>  .BI "          int (*" callback ") (struct dl_phdr_info *" info ,
> -.BI "                           size_t " size ", void *" data ,
> -.BI "          void *" data ;
> +.BI "                           size_t " size ", void *" data "),"
> +.BI "          void *" data ");"
>  .fi
>  .SH DESCRIPTION
>  The
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 10/20] inet_net_pton.3: srcfix, cfix

[Rasmus Villemoes]

Use a consistent style throughout the man-pages.
---
 man3/inet_net_pton.3 | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/man3/inet_net_pton.3 b/man3/inet_net_pton.3
index 8276012..340a849 100644
--- a/man3/inet_net_pton.3
+++ b/man3/inet_net_pton.3
@@ -37,8 +37,7 @@ inet_net_pton, inet_net_ntop \- Internet network number conversion
 .BI "                    char *" pres ", size_t " psize );
 .fi
 .sp
-Link with
-.IR \-lresolv .
+Link with \fI\-lresolv\fP.
 .sp
 .in -4n
 Feature Test Macro Requirements for glibc (see
@@ -331,7 +330,7 @@ Raw address:              c1a80180
 .in
 .SS Program source
 .nf
-/* Link with -lresolv */
+/* Link with "\-lresolv" */
 
 #include <arpa/inet.h>
 #include <stdio.h>
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 10/20] inet_net_pton.3: srcfix, cfix

[Michael Kerrisk (man-pages)]

Applied. Thanks, Rasmus.

Cheers,

Michael



On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> Use a consistent style throughout the man-pages.
> ---
>  man3/inet_net_pton.3 | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/man3/inet_net_pton.3 b/man3/inet_net_pton.3
> index 8276012..340a849 100644
> --- a/man3/inet_net_pton.3
> +++ b/man3/inet_net_pton.3
> @@ -37,8 +37,7 @@ inet_net_pton, inet_net_ntop \- Internet network number conversion
>  .BI "                    char *" pres ", size_t " psize );
>  .fi
>  .sp
> -Link with
> -.IR \-lresolv .
> +Link with \fI\-lresolv\fP.
>  .sp
>  .in -4n
>  Feature Test Macro Requirements for glibc (see
> @@ -331,7 +330,7 @@ Raw address:              c1a80180
>  .in
>  .SS Program source
>  .nf
> -/* Link with -lresolv */
> +/* Link with "\-lresolv" */
>  
>  #include <arpa/inet.h>
>  #include <stdio.h>
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 11/20] envz_add.3: fix prototypes

[Rasmus Villemoes]

The envz_len parameters for envz_entry() and envz_get() are not passed
by reference.
---
 man3/envz_add.3 | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/man3/envz_add.3 b/man3/envz_add.3
index cb85833..de388af 100644
--- a/man3/envz_add.3
+++ b/man3/envz_add.3
@@ -18,10 +18,10 @@ envz_remove, envz_strip \- environment string support
 .BI "error_t envz_add(char **" envz ", size_t *" envz_len ,
 .BI "                 const char *" name ", const char *" value );
 
-.BI "char *envz_entry(const char *" envz ", size_t *" envz_len \
+.BI "char *envz_entry(const char *" envz ", size_t " envz_len \
 ", const char *" name );
 
-.BI "char *envz_get(const char *" envz ", size_t *" envz_len \
+.BI "char *envz_get(const char *" envz ", size_t " envz_len \
 ", const char *" name );
 
 .BI "error_t envz_merge(char **" envz ", size_t *" envz_len ,
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 11/20] envz_add.3: fix prototypes

[Michael Kerrisk (man-pages)]

Thanks, Rasmus! Applied.

Cheers,

Michael


On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The envz_len parameters for envz_entry() and envz_get() are not passed
> by reference.
> ---
>  man3/envz_add.3 | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/man3/envz_add.3 b/man3/envz_add.3
> index cb85833..de388af 100644
> --- a/man3/envz_add.3
> +++ b/man3/envz_add.3
> @@ -18,10 +18,10 @@ envz_remove, envz_strip \- environment string support
>  .BI "error_t envz_add(char **" envz ", size_t *" envz_len ,
>  .BI "                 const char *" name ", const char *" value );
>  
> -.BI "char *envz_entry(const char *" envz ", size_t *" envz_len \
> +.BI "char *envz_entry(const char *" envz ", size_t " envz_len \
>  ", const char *" name );
>  
> -.BI "char *envz_get(const char *" envz ", size_t *" envz_len \
> +.BI "char *envz_get(const char *" envz ", size_t " envz_len \
>  ", const char *" name );
>  
>  .BI "error_t envz_merge(char **" envz ", size_t *" envz_len ,
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 12/20] fpathconf.3: fix prototype

[Rasmus Villemoes]

The path argument to pathconf() is const.
---
 man3/fpathconf.3 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man3/fpathconf.3 b/man3/fpathconf.3
index d8d9a1b..0e46f31 100644
--- a/man3/fpathconf.3
+++ b/man3/fpathconf.3
@@ -47,7 +47,7 @@ fpathconf, pathconf \- get configuration values for files
 .sp
 .BI "long fpathconf(int " fd ", int " name );
 .br
-.BI "long pathconf(char *" path ", int " name );
+.BI "long pathconf(const char *" path ", int " name );
 .fi
 .SH DESCRIPTION
 .BR fpathconf ()
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 12/20] fpathconf.3: fix prototype

[Michael Kerrisk (man-pages)]

Thanks, Rasmus! Applied.

Cheers,

Michael


On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The path argument to pathconf() is const.
> ---
>  man3/fpathconf.3 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man3/fpathconf.3 b/man3/fpathconf.3
> index d8d9a1b..0e46f31 100644
> --- a/man3/fpathconf.3
> +++ b/man3/fpathconf.3
> @@ -47,7 +47,7 @@ fpathconf, pathconf \- get configuration values for files
>  .sp
>  .BI "long fpathconf(int " fd ", int " name );
>  .br
> -.BI "long pathconf(char *" path ", int " name );
> +.BI "long pathconf(const char *" path ", int " name );
>  .fi
>  .SH DESCRIPTION
>  .BR fpathconf ()
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 13/20] fseek.3: fix prototype

[Rasmus Villemoes]

The pos argument to fsetpos() is const.
---
 man3/fseek.3 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man3/fseek.3 b/man3/fseek.3
index f46ee82..c22a2c2 100644
--- a/man3/fseek.3
+++ b/man3/fseek.3
@@ -53,7 +53,7 @@ fgetpos, fseek, fsetpos, ftell, rewind \- reposition a stream
 
 .BI "int fgetpos(FILE *" stream ", fpos_t *" pos );
 .br
-.BI "int fsetpos(FILE *" stream ", fpos_t *" pos );
+.BI "int fsetpos(FILE *" stream ", const fpos_t *" pos );
 .SH DESCRIPTION
 The
 .BR fseek ()
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 13/20] fseek.3: fix prototype

[Michael Kerrisk (man-pages)]

Thanks. Applied!

Cheers,

Michael


On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The pos argument to fsetpos() is const.
> ---
>  man3/fseek.3 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man3/fseek.3 b/man3/fseek.3
> index f46ee82..c22a2c2 100644
> --- a/man3/fseek.3
> +++ b/man3/fseek.3
> @@ -53,7 +53,7 @@ fgetpos, fseek, fsetpos, ftell, rewind \- reposition a stream
>  
>  .BI "int fgetpos(FILE *" stream ", fpos_t *" pos );
>  .br
> -.BI "int fsetpos(FILE *" stream ", fpos_t *" pos );
> +.BI "int fsetpos(FILE *" stream ", const fpos_t *" pos );
>  .SH DESCRIPTION
>  The
>  .BR fseek ()
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 14/20] gcvt.3: fix prototype

[Rasmus Villemoes]

The ndigit paramenter to gcvt() has type int.
---
 man3/gcvt.3 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man3/gcvt.3 b/man3/gcvt.3
index 2d7049b..77d910a 100644
--- a/man3/gcvt.3
+++ b/man3/gcvt.3
@@ -34,7 +34,7 @@ gcvt \- convert a floating-point number to a string
 .nf
 .B #include <stdlib.h>
 .sp
-.BI "char *gcvt(double " number ", size_t " ndigit ", char *" buf );
+.BI "char *gcvt(double " number ", int " ndigit ", char *" buf );
 .fi
 .sp
 .in -4n
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 14/20] gcvt.3: fix prototype

[Michael Kerrisk (man-pages)]

Thanks. Applied!

Cheers,

Michael

On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The ndigit paramenter to gcvt() has type int.
> ---
>  man3/gcvt.3 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man3/gcvt.3 b/man3/gcvt.3
> index 2d7049b..77d910a 100644
> --- a/man3/gcvt.3
> +++ b/man3/gcvt.3
> @@ -34,7 +34,7 @@ gcvt \- convert a floating-point number to a string
>  .nf
>  .B #include <stdlib.h>
>  .sp
> -.BI "char *gcvt(double " number ", size_t " ndigit ", char *" buf );
> +.BI "char *gcvt(double " number ", int " ndigit ", char *" buf );
>  .fi
>  .sp
>  .in -4n
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 15/20] getaddrinfo_a.3: fix prototype

[Rasmus Villemoes]

The pointer arguments to gai_suspend() are const.
---
 man3/getaddrinfo_a.3 | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/man3/getaddrinfo_a.3 b/man3/getaddrinfo_a.3
index 768fc97..1873307 100644
--- a/man3/getaddrinfo_a.3
+++ b/man3/getaddrinfo_a.3
@@ -38,8 +38,8 @@ network address and service translation
 .BI "int getaddrinfo_a(int " "mode" ", struct gaicb *" "list[]" ,
 .BI "                int " "nitems" ", struct sigevent *" "sevp" );
 .sp
-.BI "int gai_suspend(struct gaicb *" "list[]" ", int " "nitems" ,
-.BI "                struct timespec *" "timeout" );
+.BI "int gai_suspend(const struct gaicb * const " "list[]" ", int " "nitems" ,
+.BI "                const struct timespec *" "timeout" );
 .sp
 .BI "int gai_error(struct gaicb *" "req" );
 .sp
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 15/20] getaddrinfo_a.3: fix prototype

[Michael Kerrisk (man-pages)]

Thanks. Applied!

Cheers,

Michael


On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The pointer arguments to gai_suspend() are const.
> ---
>  man3/getaddrinfo_a.3 | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/man3/getaddrinfo_a.3 b/man3/getaddrinfo_a.3
> index 768fc97..1873307 100644
> --- a/man3/getaddrinfo_a.3
> +++ b/man3/getaddrinfo_a.3
> @@ -38,8 +38,8 @@ network address and service translation
>  .BI "int getaddrinfo_a(int " "mode" ", struct gaicb *" "list[]" ,
>  .BI "                int " "nitems" ", struct sigevent *" "sevp" );
>  .sp
> -.BI "int gai_suspend(struct gaicb *" "list[]" ", int " "nitems" ,
> -.BI "                struct timespec *" "timeout" );
> +.BI "int gai_suspend(const struct gaicb * const " "list[]" ", int " "nitems" ,
> +.BI "                const struct timespec *" "timeout" );
>  .sp
>  .BI "int gai_error(struct gaicb *" "req" );
>  .sp
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 16/20] getnameinfo.3: fix prototype

[Rasmus Villemoes]

The hostlen and servlen parameters to getnameinfo() have type
socklet_t.
---
 man3/getnameinfo.3 | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/man3/getnameinfo.3 b/man3/getnameinfo.3
index db72bd7..16c8d4f 100644
--- a/man3/getnameinfo.3
+++ b/man3/getnameinfo.3
@@ -16,8 +16,8 @@ getnameinfo \- address-to-name translation in protocol-independent manner
 .B #include <netdb.h>
 .sp
 .BI "int getnameinfo(const struct sockaddr *" "sa" ", socklen_t " "salen" ,
-.BI "                char *" "host" ", size_t " "hostlen" ,
-.BI "                char *" "serv" ", size_t " "servlen" ", int " "flags" );
+.BI "                char *" "host" ", socklen_t " "hostlen" ,
+.BI "                char *" "serv" ", socklen_t " "servlen" ", int " "flags" );
 .fi
 .sp
 .in -4n
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 16/20] getnameinfo.3: fix prototype

[Michael Kerrisk (man-pages)]

Thanks, Rasmus. It looks like this changed in glibc 2.2, with commit
e10b8512ce40cfb9f706e40edc7ed0ebdd151471.

Patch applied.

Cheers,

Michael


On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The hostlen and servlen parameters to getnameinfo() have type
> socklet_t.
> ---
>  man3/getnameinfo.3 | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/man3/getnameinfo.3 b/man3/getnameinfo.3
> index db72bd7..16c8d4f 100644
> --- a/man3/getnameinfo.3
> +++ b/man3/getnameinfo.3
> @@ -16,8 +16,8 @@ getnameinfo \- address-to-name translation in protocol-independent manner
>  .B #include <netdb.h>
>  .sp
>  .BI "int getnameinfo(const struct sockaddr *" "sa" ", socklen_t " "salen" ,
> -.BI "                char *" "host" ", size_t " "hostlen" ,
> -.BI "                char *" "serv" ", size_t " "servlen" ", int " "flags" );
> +.BI "                char *" "host" ", socklen_t " "hostlen" ,
> +.BI "                char *" "serv" ", socklen_t " "servlen" ", int " "flags" );
>  .fi
>  .sp
>  .in -4n
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 17/20] getrpcent.3: fix prototype

[Rasmus Villemoes]

The argument to getrpcbyname() is const.
---
 man3/getrpcent.3 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man3/getrpcent.3 b/man3/getrpcent.3
index 760b5ab..f25f7eb 100644
--- a/man3/getrpcent.3
+++ b/man3/getrpcent.3
@@ -15,7 +15,7 @@ RPC entry
 
 .BI "struct rpcent *getrpcent(void);"
 
-.BI "struct rpcent *getrpcbyname(char *" name );
+.BI "struct rpcent *getrpcbyname(const char *" name );
 
 .BI "struct rpcent *getrpcbynumber(int " number );
 
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 17/20] getrpcent.3: fix prototype

[Michael Kerrisk (man-pages)]

Thanks. Applied!

Cheers,

Michael


On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The argument to getrpcbyname() is const.
> ---
>  man3/getrpcent.3 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man3/getrpcent.3 b/man3/getrpcent.3
> index 760b5ab..f25f7eb 100644
> --- a/man3/getrpcent.3
> +++ b/man3/getrpcent.3
> @@ -15,7 +15,7 @@ RPC entry
>  
>  .BI "struct rpcent *getrpcent(void);"
>  
> -.BI "struct rpcent *getrpcbyname(char *" name );
> +.BI "struct rpcent *getrpcbyname(const char *" name );
>  
>  .BI "struct rpcent *getrpcbynumber(int " number );
>  
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 18/20] getspnam.3: fix prototype

[Rasmus Villemoes]

The struct spwd argument to putspent is const.
---
 man3/getspnam.3 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man3/getspnam.3 b/man3/getspnam.3
index 3fbc7d1..40d78c2 100644
--- a/man3/getspnam.3
+++ b/man3/getspnam.3
@@ -28,7 +28,7 @@ lckpwdf, ulckpwdf \- get shadow password file entry
 .sp
 .BI "struct spwd *sgetspent(const char *" s );
 .sp
-.BI "int putspent(struct spwd *" p ", FILE *" fp );
+.BI "int putspent(const struct spwd *" p ", FILE *" fp );
 .sp
 .B int lckpwdf(void);
 .sp
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 18/20] getspnam.3: fix prototype

[Michael Kerrisk (man-pages)]

Thanks. Applied!

Cheers,

Michael


On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The struct spwd argument to putspent is const.
> ---
>  man3/getspnam.3 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man3/getspnam.3 b/man3/getspnam.3
> index 3fbc7d1..40d78c2 100644
> --- a/man3/getspnam.3
> +++ b/man3/getspnam.3
> @@ -28,7 +28,7 @@ lckpwdf, ulckpwdf \- get shadow password file entry
>  .sp
>  .BI "struct spwd *sgetspent(const char *" s );
>  .sp
> -.BI "int putspent(struct spwd *" p ", FILE *" fp );
> +.BI "int putspent(const struct spwd *" p ", FILE *" fp );
>  .sp
>  .B int lckpwdf(void);
>  .sp
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 19/20] getutent.3: fix prototypes

[Rasmus Villemoes]

The arguments to getutid(), getutline() and pututline() are const.
---
 man3/getutent.3 | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/man3/getutent.3 b/man3/getutent.3
index 6c7ff1b..5f5ac68 100644
--- a/man3/getutent.3
+++ b/man3/getutent.3
@@ -37,11 +37,11 @@ utmpname \- access utmp file entries
 .sp
 .B struct utmp *getutent(void);
 .br
-.BI "struct utmp *getutid(struct utmp *" ut );
+.BI "struct utmp *getutid(const struct utmp *" ut );
 .br
-.BI "struct utmp *getutline(struct utmp *" ut );
+.BI "struct utmp *getutline(const struct utmp *" ut );
 .sp
-.BI "struct utmp *pututline(struct utmp *" ut );
+.BI "struct utmp *pututline(const struct utmp *" ut );
 .sp
 .B void setutent(void);
 .br
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 19/20] getutent.3: fix prototypes

[Michael Kerrisk (man-pages)]

Thanks. Applied!

Cheers,

Michael


On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The arguments to getutid(), getutline() and pututline() are const.
> ---
>  man3/getutent.3 | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/man3/getutent.3 b/man3/getutent.3
> index 6c7ff1b..5f5ac68 100644
> --- a/man3/getutent.3
> +++ b/man3/getutent.3
> @@ -37,11 +37,11 @@ utmpname \- access utmp file entries
>  .sp
>  .B struct utmp *getutent(void);
>  .br
> -.BI "struct utmp *getutid(struct utmp *" ut );
> +.BI "struct utmp *getutid(const struct utmp *" ut );
>  .br
> -.BI "struct utmp *getutline(struct utmp *" ut );
> +.BI "struct utmp *getutline(const struct utmp *" ut );
>  .sp
> -.BI "struct utmp *pututline(struct utmp *" ut );
> +.BI "struct utmp *pututline(const struct utmp *" ut );
>  .sp
>  .B void setutent(void);
>  .br
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 20/20] getrpcport.3: add #include and fix prototype

[Rasmus Villemoes]

The prototype for getrpcport() is obtained by #include'ing
<rpc/rpc.h>. Also, update its prototype.
---
 man3/getrpcport.3 | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/man3/getrpcport.3 b/man3/getrpcport.3
index 16ccdfc..a31fbc9 100644
--- a/man3/getrpcport.3
+++ b/man3/getrpcport.3
@@ -10,8 +10,10 @@
 getrpcport \- get RPC port number
 .SH SYNOPSIS
 .nf
-.BI "int getrpcport(char *" host ", int " prognum ", int " versnum \
-", int " proto );
+.B "#include <rpc/rpc.h>"
+.sp
+.BI "int getrpcport(const char *" host ", unsigned long " prognum \
+", unsigned long " versnum ", unsigned " proto );
 .fi
 .SH DESCRIPTION
 .BR getrpcport ()
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 20/20] getrpcport.3: add #include and fix prototype

[Michael Kerrisk (man-pages)]

Thanks, rasmus. Applied.

Cheers,

Michael



On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The prototype for getrpcport() is obtained by #include'ing
> <rpc/rpc.h>. Also, update its prototype.
> ---
>  man3/getrpcport.3 | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/man3/getrpcport.3 b/man3/getrpcport.3
> index 16ccdfc..a31fbc9 100644
> --- a/man3/getrpcport.3
> +++ b/man3/getrpcport.3
> @@ -10,8 +10,10 @@
>  getrpcport \- get RPC port number
>  .SH SYNOPSIS
>  .nf
> -.BI "int getrpcport(char *" host ", int " prognum ", int " versnum \
> -", int " proto );
> +.B "#include <rpc/rpc.h>"
> +.sp
> +.BI "int getrpcport(const char *" host ", unsigned long " prognum \
> +", unsigned long " versnum ", unsigned " proto );
>  .fi
>  .SH DESCRIPTION
>  .BR getrpcport ()
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 00/20] More prototype fixes

[Rasmus Villemoes]

Sorry, I forgot the Signed-off-by tags. Do you need those?

Rasmus
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 00/20] More prototype fixes

[Michael Kerrisk (man-pages)]

On 05/20/2014 01:58 AM, Rasmus Villemoes wrote:
> Sorry, I forgot the Signed-off-by tags. Do you need those?

No, it's okay for these patches, since changes are trivial.

Cheers,

Michael


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 00/20] More prototype fixes

[Michael Kerrisk (man-pages)]

On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The previously mentioned perl script can also be applied to the man3
> pages. This round of patches comes mostly from applying it to
> man3/[a-g].*, but I've also included other stuff I've stumbled across.

Thanks, Rasmus. This looks mostly good at a quick scan. I'll start 
working through these and apply them in a branch (that I will merge 
soon).

Cheers,

Michael


> The perl script itself follows soon.
> 
> Rasmus Villemoes (20):
>   open.2: spfix
>   set_mempolicy.2: Fix prototype for set_mempolicy(2)
>   remap_file_pages.2: Fix prototype
>   swapon.2: remove header from synopsis
>   a64l.3: fix prototype for a64l()
>   argz_*: Fix prototypes
>   adjtime.3: add required header
>   bstring.3: Fix prototypes
>   dl_iterate_phdr.3: tfix
>   inet_net_pton.3: srcfix, cfix
>   envz_add.3: fix prototypes
>   fpathconf.3: fix prototype
>   fseek.3: fix prototype
>   gcvt.3: fix prototype
>   getaddrinfo_a.3: fix prototype
>   getnameinfo.3: fix prototype
>   getrpcent.3: fix prototype
>   getspnam.3: fix prototype
>   getutent.3: fix prototypes
>   getrpcport.3: add #include and fix prototype
> 
>  man2/open.2             | 2 +-
>  man2/remap_file_pages.2 | 2 +-
>  man2/set_mempolicy.2    | 4 ++--
>  man2/swapon.2           | 2 --
>  man3/a64l.3             | 2 +-
>  man3/adjtime.3          | 2 ++
>  man3/argz_add.3         | 6 +++---
>  man3/bstring.3          | 6 +++---
>  man3/dl_iterate_phdr.3  | 4 ++--
>  man3/envz_add.3         | 4 ++--
>  man3/fpathconf.3        | 2 +-
>  man3/fseek.3            | 2 +-
>  man3/gcvt.3             | 2 +-
>  man3/getaddrinfo_a.3    | 4 ++--
>  man3/getnameinfo.3      | 4 ++--
>  man3/getrpcent.3        | 2 +-
>  man3/getrpcport.3       | 6 ++++--
>  man3/getspnam.3         | 2 +-
>  man3/getutent.3         | 6 +++---
>  man3/inet_net_pton.3    | 5 ++---
>  20 files changed, 35 insertions(+), 34 deletions(-)
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 00/20] More prototype fixes

[Michael Kerrisk (man-pages)]

On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
> The previously mentioned perl script can also be applied to the man3
> pages. This round of patches comes mostly from applying it to
> man3/[a-g].*, but I've also included other stuff I've stumbled across.

Thanks for all these patches, Rasmus. I'll keep them parked in 
branch for the moment. 

One small note for future patches. You've git the subject lines 
almost exactly how I prefer, but if you could capitalize the first 
word following the page name in the subject line, then that would 
be perfect. So, for example, instead of

    fseek.3: fix prototype

use 

    fseek.3: Fix prototype

Thanks,

Michael



> The perl script itself follows soon.
> 
> Rasmus Villemoes (20):
>   open.2: spfix
>   set_mempolicy.2: Fix prototype for set_mempolicy(2)
>   remap_file_pages.2: Fix prototype
>   swapon.2: remove header from synopsis
>   a64l.3: fix prototype for a64l()
>   argz_*: Fix prototypes
>   adjtime.3: add required header
>   bstring.3: Fix prototypes
>   dl_iterate_phdr.3: tfix
>   inet_net_pton.3: srcfix, cfix
>   envz_add.3: fix prototypes
>   fpathconf.3: fix prototype
>   fseek.3: fix prototype
>   gcvt.3: fix prototype
>   getaddrinfo_a.3: fix prototype
>   getnameinfo.3: fix prototype
>   getrpcent.3: fix prototype
>   getspnam.3: fix prototype
>   getutent.3: fix prototypes
>   getrpcport.3: add #include and fix prototype
> 
>  man2/open.2             | 2 +-
>  man2/remap_file_pages.2 | 2 +-
>  man2/set_mempolicy.2    | 4 ++--
>  man2/swapon.2           | 2 --
>  man3/a64l.3             | 2 +-
>  man3/adjtime.3          | 2 ++
>  man3/argz_add.3         | 6 +++---
>  man3/bstring.3          | 6 +++---
>  man3/dl_iterate_phdr.3  | 4 ++--
>  man3/envz_add.3         | 4 ++--
>  man3/fpathconf.3        | 2 +-
>  man3/fseek.3            | 2 +-
>  man3/gcvt.3             | 2 +-
>  man3/getaddrinfo_a.3    | 4 ++--
>  man3/getnameinfo.3      | 4 ++--
>  man3/getrpcent.3        | 2 +-
>  man3/getrpcport.3       | 6 ++++--
>  man3/getspnam.3         | 2 +-
>  man3/getutent.3         | 6 +++---
>  man3/inet_net_pton.3    | 5 ++---
>  20 files changed, 35 insertions(+), 34 deletions(-)
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 00/20] More prototype fixes

[Michael Kerrisk (man-pages)]

On 05/20/2014 09:09 AM, Michael Kerrisk (man-pages) wrote:
> On 05/20/2014 01:02 AM, Rasmus Villemoes wrote:
>> The previously mentioned perl script can also be applied to the man3
>> pages. This round of patches comes mostly from applying it to
>> man3/[a-g].*, but I've also included other stuff I've stumbled across.
> 
> Thanks for all these patches, Rasmus. I'll keep them parked in 
> branch for the moment. 

I meant to add, I've parked them in a branch, since I suspect you 
have more patches coming soon ;-).

Cheers,

Michael






--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html