public inbox for linux-man@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] open.2, prctl.2: SYNOPSIS: Document these as variadic functions
@ 2023-05-19 13:15 Alejandro Colomar
  0 siblings, 0 replies; only message in thread
From: Alejandro Colomar @ 2023-05-19 13:15 UTC (permalink / raw)
  To: linux-man; +Cc: Alejandro Colomar, Florian Weimer

I remember I discussed this with Michael Kerrisk a long ago and we
agreed to apply this fix, as I felt that using overload syntax was
confusing (especially since C doesn't allow overloads), but then I
didn't feel urged to write a patch.  Florian confirmed recently that
this is confusing to more programmers, so let's do it.

Reported-by: Florian Weimer <fweimer@redhat.com>
Link: <https://bugzilla.kernel.org/show_bug.cgi?id=216876#c1>
Signed-off-by: Alejandro Colomar <alx@kernel.org>
---
 man2/open.2  | 11 +++++------
 man2/prctl.2 |  5 +++--
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/man2/open.2 b/man2/open.2
index f79330a17..70b2519d8 100644
--- a/man2/open.2
+++ b/man2/open.2
@@ -38,18 +38,17 @@ Standard C library
 .nf
 .B #include <fcntl.h>
 .PP
-.BI "int open(const char *" pathname ", int " flags );
-.BI "int open(const char *" pathname ", int " flags ", mode_t " mode );
+.BI "int open(const char *" pathname ", int " flags ", ..."
+.BI "           \fR/*\fP mode_t " mode " \fR*/\fP );"
 .PP
 .BI "int creat(const char *" pathname ", mode_t " mode );
 .PP
-.BI "int openat(int " dirfd ", const char *" pathname ", int " flags );
-.BI "int openat(int " dirfd ", const char *" pathname ", int " flags \
-", mode_t " mode );
+.BI "int openat(int " dirfd ", const char *" pathname ", int " flags ", ..."
+.BI "           \fR/*\fP mode_t " mode " \fR*/\fP );"
 .PP
 /* Documented separately, in \fBopenat2\fP(2): */
 .BI "int openat2(int " dirfd ", const char *" pathname ,
-.BI "            const struct open_how *" how ", size_t " size ");"
+.BI "           const struct open_how *" how ", size_t " size ");"
 .fi
 .PP
 .RS -4
diff --git a/man2/prctl.2 b/man2/prctl.2
index 88d94b7e8..09e9072fa 100644
--- a/man2/prctl.2
+++ b/man2/prctl.2
@@ -46,8 +46,9 @@ Standard C library
 .nf
 .B #include <sys/prctl.h>
 .PP
-.BI "int prctl(int " option ", unsigned long " arg2 ", unsigned long " arg3 ,
-.BI "          unsigned long " arg4 ", unsigned long " arg5 );
+.BI "int prctl(int " option ", ..."
+.BI "          \fR/*\fP unsigned long " arg2 ", unsigned long " arg3 ,
+.BI "          unsigned long " arg4 ", unsigned long " arg5 " \fR*/\fP );"
 .fi
 .SH DESCRIPTION
 .BR prctl ()
-- 
2.40.1


^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2023-05-19 13:16 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-05-19 13:15 [PATCH] open.2, prctl.2: SYNOPSIS: Document these as variadic functions Alejandro Colomar

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox