recv.2: Mention SOCK_SEQPACKET in MSG_TRUNC flag description

unix_seqpacket_recvmsg() calls unix_dgram_recvmsg() which handles
MSG_TRUNC. This has been the case since the handling was added in
9f6f9af7694ede6314bed281eec74d588ba9474f; see net/unix/af_unix.c:

static int unix_seqpacket_recvmsg([...])
{
        [...]
return unix_dgram_recvmsg(iocb, sock, msg, size, flags);
}

The sequential-packet socket type seems to have been left out from the
description by an oversight.

Signed-off-by: Vladislav Ivanishin <vlad@ispras.ru>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

install-man.mk: Update copyright year

Signed-off-by: Alejandro Colomar <alx@kernel.org>

install-man.mk, src.mk: Respect user-specified man dirs

If a user specifies man3dir=/.../man3c, respect it and install there.
Currently, we were transforming link pages to use that dir name, but the
install location itself was still being calculated, which generated
inconsistently installed pages.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

install-man.mk: Fix link pages when installing in different mandirs

If downstream wants to put pages in different places (e.g., Debian uses
man2/ and man3/, rather than man2type/ and man3const/, man3head/, and
man3type/), make it easy for them.  Link pages need to be fixed
according to the dirname where the pages are actually being installed.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

install-man.mk: Add missing variable $man3constext

Signed-off-by: Alejandro Colomar <alx@kernel.org>

clone.2: Note EINVAL when exit_signal + bad flags

Document that Linux will report EINVAL when exit_signal is specified and
either CLONE_THREAD or CLONE_PARENT is specified.

From clone3_args_valid in Linux:
```
if ((kargs->flags & (CLONE_THREAD | CLONE_PARENT)) &&
    kargs->exit_signal)
return false;
```

I have verified that this happens on my kernel with a small program:

```
 #include <stdio.h>
 #include <linux/sched.h>
 #include <signal.h>
 #include <sys/syscall.h>
 #include <unistd.h>

int main(void)
{
struct clone_args ca = {
.flags = CLONE_THREAD | CLONE_SIGHAND | CLONE_VM,
.exit_signal = SIGCHLD, // comment me out to fix error
.set_tid_size = 0,
};
syscall(SYS_clone3, &ca, sizeof(struct clone_args));
perror("");
}
```

And I have verified that this doesn't happen with normal `clone` through
the glibc helper:

```
 #define _GNU_SOURCE

 #include <sched.h>
 #include <signal.h>
 #include <stdio.h>
 #include <sys/mman.h>

int do_nothing(void *_) { return 0; }

int main(void)
{
        void *map = mmap(NULL, 0x10000, PROT_READ | PROT_WRITE,
                 MAP_ANONYMOUS | MAP_PRIVATE, 0, 0);
void *stack_top = map + 0x10000 - 1;
clone(do_nothing, stack_top,
      CLONE_THREAD | CLONE_VM | CLONE_SIGHAND | SIGCHLD, NULL);
perror("");
}
```

Signed-off-by: Jack Pearson <jack@pearson.onl>
Cc: "Carlos O'Donell" <carlos@redhat.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

ptrace.2: Add details about usage of PTRACE_GET_SYSCALL_INFO

Document the role of PTRACE_O_TRACESYSGOOD option in connection with
PTRACE_GET_SYSCALL_INFO.

Came upon this after writing a test program using
PTRACE_GET_SYSCALL_INFO.  After failing to find what's wrong I posted a
StackOverflow question which you can find right here:
<https://stackoverflow.com/questions/72410182/ptrace-get-syscall-info-always-returns-info-op-as-ptrace-syscall-info-none>

Nate Eldredge found out what happens by looking into the kernel's source
code, here is a link to the relevant part
<https://github.com/torvalds/linux/blob/8291eaafed36f575f23951f3ce18407f480e9ecf/kernel/ptrace.c#L1018>

In the code it can be seen that in case of system call entry or exit
stops, the union is filled if and only if the signal matches
`SIGTRAP | 0x80`, a signal which is only sent if the
PTRACE_O_TRACESYSGOOD option is set.  You can read about that in the
PTRACE_O_TRACESYSGOOD section of ptrace(2)'s manual.

Complements: fc91449cb "ptrace.2: Document PTRACE_GET_SYSCALL_INFO"
Cowritten-by: Dmitry V. Levin <ldv@strace.io>
Signed-off-by: Dmitry V. Levin <ldv@strace.io>
Signed-off-by: Fotios Valasiadis <fvalasiad@gmail.com>
Acked-by: Nate Eldredge <nate@thatsmathematics.com>
Cc: Elvira Khabirova <lineprinter0@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

suffixes.7: Add .js and .ts extensions

Signed-off-by: Alejandro Colomar <alx@kernel.org>

malloc.3: EXAMPLES: Add example program with mallocarray() and macros

mallocarray() is safer than malloc(3), since it checks for overflow; it
should be preferred almost always (with the exception of non-arrays
maybe).

The macros like MALLOCARRAY() --and MALLOC()-- that perform automatic
casting and sizeof() are also safer than calling the functions directly:

-  The type of the allocated object (not the pointer) is specified as an
   argument, which improves readability:
   -  It is directly obvious what is the type of the object just by
      reading the macro call.
   -  It allows grepping for all allocations of a given type.

   This is admittedly similar to using sizeof() to get the size of the
   object, but we'll see why this is better.

-  In the case of reallocation macros, an extra check is performed to
   make sure that the previous pointer was compatible with the allocated
   type, which can avoid some mistakes.

-  The cast is performed automatically, with a pointer type derived from
   the type of the object.  This is the best point of this macro, since
   it does an automatic cast, where there's no chance of typos.

   Usually, programmers have to decide whether to cast or not the result
   of malloc(3).  Casts usually hide warnings, so are to be avoided.
   However, these functions already return a void *, so a cast doesn't
   really add much danger.  Moreover, a cast can even add warnings in
   this exceptional case, if the type of the cast is different than the
   type of the assigned pointer.  Performing a manual cast is still not
   perfect, since there are chances that a mistake will be done, and
   even ignoring accidents, they clutter code, hurting readability.
   And now we have a cast that is synced with sizeof.

-  Whenever the type of the object changes, since we perform an explicit
   cast to the old type, there will be a warning due to type mismatch in
   the assignment, so we'll be able to see all lines that are affected
   by such a change.  This is especially important, since changing the
   type of a variable and missing to update an allocation call far away
   from the declaration is easy, and the consequences can be quite bad

Apart from those benefits, there are other minor style benefits:

-  Consistency in getting the size of the object from sizeof(type),
   instead of a mix of sizeof(type) sometimes and sizeof(*p) other
   times.

-  More readable code: no casts, and no sizeof(), so also shorter lines
   that we don't need to cut.

-  Consistency in using array allocation calls for allocations of arrays
   of objects, even when the object size is 1.

Link: <https://github.com/shadow-maint/shadow/pull/649>
Cc: Serge Hallyn <serge@hallyn.com>
Cc: Iker Pedrosa <ipedrosa@redhat.com>
Cc: "Valentin V. Bartenev" <vbartenev@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

setpgid.2: ERRORS: Add EPERM for nonexisting process group

Reproduced using test program:

 #include <stdio.h>
 #include <sys/types.h>
 #include <unistd.h>

int main(void) {
pid_t  pid_a, pid_b;

pid_a = fork();
printf("pid: %i; pgid: %i; forkA: %i\n",
       (int) getpid(), (int) getpgrp(), (int) pid_a);
if (!pid_a) {
// pid_a is not a valid process group
// setpgid(0, 0);  // This makes it succeed.
sleep(1); // stay alive
return 0;
}
pid_b = fork();
if (!pid_b) {
printf("* pid: %i; pgid: %i; forkA: %i; forkB: %i\n",
       (int)getpid(), (int)getpgrp(), (int)pid_a, (int)pid_b);
setpgid(0, pid_a);
perror("setpgid");
return 0;
}
printf("pid: %i; pgid: %i; forkA: %i; forkB: %i\n",
       (int) getpid(), (int) getpgrp(), (int) pid_a, (int) pid_b);
sleep(1); // stay alive
return 0;
}

Signed-off-by: Guy Shefy <guyshefyb@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

landlock.7: ffix

Signed-off-by: Alejandro Colomar <alx@kernel.org>

landlock.7: Move the warning about missing features into the CAVEATS section

Putting the warning there makes it more prominent.
CAVEATS is a standard section that exists in many man pages
and is also described in man-pages(7).

Signed-off-by: Alejandro Colomar <alx@kernel.org>

sscanf.3: RETURN VALUE: These functions don't use a stream

This text I forgot to remove it when the page was split from scanf(3).

Signed-off-by: Alejandro Colomar <alx@kernel.org>

printf.h.3head: ffix

Signed-off-by: Alejandro Colomar <alx@kernel.org>

memcmp.3: wfix

Link: <https://lore.kernel.org/linux-man/27264e6b-bc50-f772-f8d5-1abc4ebcbe62@gmail.com/T/>
Signed-off-by: Tom Schwindl <schwindl@posteo.de>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

Start of man-pages-NEXT: Move Changes to Changes.old

Signed-off-by: Alejandro Colomar <alx@kernel.org>

lsm: Released 6.03

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Changes: Ready for 6.03

Signed-off-by: Alejandro Colomar <alx@kernel.org>

_Generic.3: Remove example code

Casting sockaddr structures is just a symptom that these APIs were
seriously misdesigned.  In GNU C, there's already a better way to handle
this (see [[gnu::transparent_union]]).  ISO C should be fixed.  Let's
not promote this kind of code.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

printf.h.3head: ffix

Signed-off-by: Alejandro Colomar <alx@kernel.org>

man2/: use consistent closed interval notation for value ranges

Signed-off-by: Alejandro Colomar <alx@kernel.org>

PA_CHAR.3const, PA_DOUBLE.3const, PA_FLAG_LONG.3const, PA_FLAG_LONG_DOUBLE.3const, PA_FLAG_LONG_LONG.3const, PA_FLAG_PTR.3const, PA_FLAG_SHORT.3const, PA_FLOAT.3const, PA_INT.3const, PA_LAST.3const, PA_POINTER.3const, PA_STRING.3const, PA_WCHAR.3const, PA_WSTRING.3const: Add links to printf.h(3head)

Cc: Walter Harms <wharms@bfs.de>
Cc: <Radisson97@gmx.de>
Cc: "G. Branden Robinson" <g.branden.robinson@gmail.com>
Cc: <libc-alpha@sourceware.org>
Signed-off-by: Alex Colomar <alx.manpages@gmail.com>

printf_arginfo_size_function.3type, printf_function.3type, printf_info.3type, printf_va_arg_function.3type: Add links to printf.h(3head)

Cc: Walter Harms <wharms@bfs.de>
Cc: <Radisson97@gmx.de>
Cc: "G. Branden Robinson" <g.branden.robinson@gmail.com>
Cc: <libc-alpha@sourceware.org>
Signed-off-by: Alex Colomar <alx.manpages@gmail.com>

register_printf_specifier.3, register_printf_modifier.3, register_printf_type.3: Add links to printf.h(3head)

Cc: Walter Harms <wharms@bfs.de>
Cc: <Radisson97@gmx.de>
Cc: "G. Branden Robinson" <g.branden.robinson@gmail.com>
Cc: <libc-alpha@sourceware.org>
Signed-off-by: Alex Colomar <alx.manpages@gmail.com>

printf.3head: Document functions for customizing printf(3)-like functions

Suggested-by: Walter Harms <wharms@bfs.de>
Cc: <Radisson97@gmx.de>
Cc: "G. Branden Robinson" <g.branden.robinson@gmail.com>
Cc: <libc-alpha@sourceware.org>
Signed-off-by: Alex Colomar <alx.manpages@gmail.com>

units.7: srcfix (\(mc -> \[mc])

Use correct *roff special character for micro sign.

Signed-off-by: G. Branden Robinson <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

adjtimex.2: srcfix

* Stop manipulating adjustment and hyphenation around a table.
* These could be safely done within the text block, but even that is not
  necessary, since what is in the next block is a single word, so no
  adjustment will take place, and to prevent hyphenation of a single
  word, \% is cheap and straightforward.  So do that.

Signed-off-by: G. Branden Robinson <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

localedef.1: srcfix

Break input lines after commas and semicolons.

Signed-off-by: G. Branden Robinson <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

localedef.1: ffix

Rewrite synopses to use groff man(7) `SY`/`YS` extension macros.

Signed-off-by: G. Branden Robinson <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

localedef.1: ffix

Stop manipulating adjustment and hyphenation.

Forcibly re-enabling adjustment to both margins after the synopsis does
not respect user configuration of adjustment.  There _is_ a portable way
to save the adjustment mode, via the .j register and a copy of it, but
doing so requires even more usage of low-level requests that are
discouraged in man page writing.

The latter is incorrect for use with groff(1) since '.hy' does not
restore the previous hyphenation mode but sets it to 1, which is not
appropriate for the English-language hyphenation patterns groff uses.
(Also, AT&T man(7) used a hyphenation mode of 14.)

Neither of these requests is respectful of user configuration of
adjustment or hyphenation enablement.  Features in the forthcoming
groff 1.23 will make these easier for users to manipulate to their
preference.

(mandoc(1) does not support configurable adjustment or hyphenation, so
all of these requests are no-ops for it.)

Signed-off-by: G. Branden Robinson <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

vdso.7: fix deadlinks to Linux Documentation/

Respectively:
commit dc7a12bdfccd94c31f79e294f16f7549bd411b49 ("docs: arm: convert
docs to ReST and rename to *.rst")
commit db9a0975a20c1f21c108b9d44545792d790593e4 ("docs: ia64: convert to
ReST")
commit e77e9187ae1caf2d83dd5e7f0c1466254b644a4c ("docs: parisc: convert
to ReST and add to documentation body")

Signed-off-by: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

times.2: ffix

Use \[lq] and \[rq] special characters for prose quotation marks.

Signed-off-by: G. Branden Robinson <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

times.2: srcfix

Break input lines after commas.

Signed-off-by: G. Branden Robinson <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

quotactl.2: ffix

Use \- for minus sign instead of - (hyphen), producing better typography
on typesetting devices and Unicode terminals.

Signed-off-by: G. Branden Robinson <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

close_range.2: srcfix (~ -> \[ti])

Use correct *roff special character for ("ASCII") tilde.

There are other occurrences in bpf-helpers.7, but those will have to
await improvement of the tool that generates man(7) from the source
language.

Signed-off-by: G. Branden Robinson <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

man*/: srcfix (^ -> \[ha])

Use correct *roff special character for hat/caret/circumflex accent.

Signed-off-by: G. Branden Robinson <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

ioctl_fat.2: srcfix

Remove unportable hack to put a dot at the beginning of an input line.
Use the idiomatic means of doing this instead: the roff dummy character
escape sequence.

Signed-off-by: G. Branden Robinson <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

prctl.2: srcfix

Remove spurious escaping of '.' where it cannot be interpreted as a roff
control character.

Signed-off-by: G. Branden Robinson <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

sched.7: srcfix

Fix warnings from formatter.

troff:./man7/sched.7:989: warning [p 10, 2.7i]: cannot adjust line
troff:./man7/sched.7:990: warning [p 10, 2.8i]: cannot adjust line

Add hyphenless break points to file names, but also suppress hyphenation
to reduce copy-and-paste frustration.

Signed-off-by: G. Branden Robinson <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

man2/: srcfix (hyphens used as C operators)

Escape hyphens used as parts of C `->` operators.

Signed-off-by: G. Branden Robinson <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

getrandom.2: Change limit to use IEC multiple symbol

Signed-off-by: Alejandro Colomar <alx@kernel.org>

reboot.2: Show BCD dates in hex not decimal

Signed-off-by: Alejandro Colomar <alx@kernel.org>

spu_run.2: Fix example comment status code or-ed value

Signed-off-by: Alejandro Colomar <alx@kernel.org>

shmget.2: Fix limit units prefix symbol from SI to IEC

Signed-off-by: Alejandro Colomar <alx@kernel.org>

perf_event_open.2: Add missing variable name

Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

persistent-keyring.7: wfix

Reported-by: Helge Kreutzmann <debian@helgefjell.de>
Signed-off-by: Jakub Wilk <jwilk@jwilk.net>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

units.7: Add break points in URI

Signed-off-by: Alejandro Colomar <alx@kernel.org>

units.7: Fix escapes (\-) in URI

Signed-off-by: Alejandro Colomar <alx@kernel.org>

units.7: Use \[mc] instead of \(mc

Signed-off-by: Alejandro Colomar <alx@kernel.org>

prctl.2, man.7: Use \[ga] instead of \(ga

Signed-off-by: Alejandro Colomar <alx@kernel.org>

prctl.2: Use \[cq] instead of \(cq

Signed-off-by: Alejandro Colomar <alx@kernel.org>

prctl.2: Use \[oq] instead of \(oq

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Many pages: Use \[bu] instead of \(bu

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Various pages: Use \[rq] instead of \(rq

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Various pages: Use \[lq] instead of \(lq

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Many pages: Use \[ha] instead of \(ha

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Many pages: Use \[ti] instead of \(ti

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Many pages: Use \[em] instead of \(em

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Many pages: Use \[en] instead of \(en

Signed-off-by: Alejandro Colomar <alx@kernel.org>

man-pages.7: Recommend using \[..] instead of \(.. escapes

They are more readable.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

EOF.3const: ffix

Reported-by: наб <nabijaczleweli@nabijaczleweli.xyz>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

timespec.3type: tv_nsec is impl-def-type, glibc llong not a bug

n3091 accepts n3066, making it part of the next working draft and C23:
  https://www.open-std.org/jtc1/sc22/wg14/www/docs/n3091.doc

Update timespec.3type appropriately, largely mirroring my paper.

Signed-off-by: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

Many pages: Use \[dq] instead of \(dq

This improves readability in the source code, since it delimits where
the escape sequence ends.

Cc: наб <nabijaczleweli@nabijaczleweli.xyz>
Cc: "G. Branden Robinson" <g.branden.robinson@gmail.com>
Cc: Brian Inglis <Brian.Inglis@Shaw.ca>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

Many pages: Use \[aq] instead of \(aq

This improves readability in the source code, since it delimits where
the escape sequence ends.

Cc: наб <nabijaczleweli@nabijaczleweli.xyz>
Cc: "G. Branden Robinson" <g.branden.robinson@gmail.com>
Cc: Brian Inglis <Brian.Inglis@Shaw.ca>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

Many pages: Use lowercase for 'glibc'

It's a proper noun, whose original letter case should be respected.
glibc's own documentation uses always lowercase; let's do the same here.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

sscanf.3: BUGS: Document the UB in some conversion specifiers

This is a bug in the standards, but implementation should not follow the
standard in this case.

Link: <https://lore.kernel.org/linux-man/20221208123454.13132-1-abbotti@mev.co.uk/T/#u>
Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Zack Weinberg <zack@owlfolio.org>
Cc: Joseph Myers <joseph@codesourcery.com>
Cc: Eric Biggers <ebiggers@kernel.org>
Cc: "G. Branden Robinson" <g.branden.robinson@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

namespaces.7: ffix

Reported-by: Helge Kreutzmann <debian@helgefjell.de>
Cc: "G. Branden Robinson" <g.branden.robinson@gmail.com>
Cc: Mario Blaettermann <mario.blaettermann@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

regex.3: tfix

Reported-by: Helge Kreutzmann <debian@helgefjell.de>
Cc: "G. Branden Robinson" <g.branden.robinson@gmail.com>
Cc: Mario Blaettermann <mario.blaettermann@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

charsets.7: wfix

Reported-by: Helge Kreutzmann <debian@helgefjell.de>
Cc: Mario Blaettermann <mario.blaettermann@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

perf_event_open.2: Update recent changes

Add missing perf_event_attr fields, new event codes and sample type.
Also add descriptions for PERF_FORMAT_LOST.

Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

landlock.7: tfix

Signed-off-by: Günther Noack <gnoack3000@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

landlock.7: ffix

Signed-off-by: Günther Noack <gnoack3000@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

vdso.7: Fix risc-v symbol names.

The kernel git history says the names have always been "__vdso_" rather
than "__kernel_", so I assume this was a copy & paste mistake from a
different architecture.

Luckily, the path to the kernel source that lets you confirm/deny this
_is_ correct :-)

Signed-off-by: Elliott Hughes <enh@google.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

bpf-helpers.7: Refresh page

See commit 19c7f78393f2 ("bpf-helpers.7: Refresh page")

Signed-off-by: Alejandro Colomar <alx@kernel.org>

intN_t.3type: tfix

Signed-off-by: Jakub Wilk <jwilk@jwilk.net>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

charsets.7: wfix

Several of the languages mentioned are not West Europe.  Some come from
the North, South, or East.

Reported-by: Helge Kreutzmann <debian@helgefjell.de>
Cc: Mario Blaettermann <mario.blaettermann@gmail.com>
Cc: Stefan Puiu <stefan.puiu@gmail.com>
Cc: Bernd Petrovitsch <bernd@petrovitsch.priv.at>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

cppcheck.suppress, lint-c.mk: lint-c-cppcheck: Add cppcheck(1) to the C linters

Signed-off-by: Alejandro Colomar <alx@kernel.org>

getdents.2: EXAMPLES: Fix printf() conversion specifier

Signed-off-by: Alejandro Colomar <alx@kernel.org>

fopencookie.3: EXAMPLES: Fix memory leak

Signed-off-by: Alejandro Colomar <alx@kernel.org>

seccomp_unotify.2: EXAMPLES: tfix

Signed-off-by: Alejandro Colomar <alx@kernel.org>

getdate.3: EXAMPLES: Fix printf() conversion specifier

Fixes: b42296e4 "Various pages: EXAMPLES: Use unsigned types for loop iterators"
Signed-off-by: Alejandro Colomar <alx@kernel.org>

pthread_setschedparam.3: EXAMPLES: Don't assign unused value

We're not reading the value after it's set.  And I just checked that
that function can't fail for reasonable input.

Reported-by: cppcheck(1)
Reported-by: `make lint-c-cppcheck`
Signed-off-by: Alejandro Colomar <alx@kernel.org>

tsearch.3: EXAMPLES: Don't else after noreturn

Signed-off-by: Alejandro Colomar <alx@kernel.org>

stpncpy.3: EXAMPLES: tfix

Signed-off-by: Alejandro Colomar <alx@kernel.org>

rpmatch.3: EXAMPLES: Use _DEFAULT_SOURCE instead of _SVID_SOURCE

_SVID_SOURCE is deprecated.  Show how a program should be written today.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

static_assert.3: srcfix

Add missing ." SRC END comment.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

build-src.mk: CFLAGS: Add -Wdeclaration-after-statement

This helps write more readable code, separating variable declarations
from code.  In some cases, when initializing structs, or declaring some
VLAs, we can't follow the rule, so don't make it an error.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Various pages: EXAMPLES: Fix -Wdeclaration-after-statement errors

Signed-off-by: Alejandro Colomar <alx@kernel.org>

Various pages: wfix

Fixes: b324e17d3208 ("Many pages: wfix")
Reported-by: Reported-by: Helge Kreutzmann <debian@helgefjell.de>
Cc: Mario Blaettermann <mario.blaettermann@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

getpid.2: tfix

Reported-by: Reported-by: Helge Kreutzmann <debian@helgefjell.de>
Cc: Mario Blaettermann <mario.blaettermann@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

_exit.2: tfix

Reported-by: Reported-by: Helge Kreutzmann <debian@helgefjell.de>
Cc: Mario Blaettermann <mario.blaettermann@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

ttytype.5: ffix

Reported-by: Helge Kreutzmann <debian@helgefjell.de>
Cc: Mario Blaettermann <mario.blaettermann@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

user_namespaces.7: ffix

Reported-by: Helge Kreutzmann <debian@helgefjell.de>
Cc: Mario Blaettermann <mario.blaettermann@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

string.3: wfix

Use consistent syntax for 'dest' vs 'dst' in string(3).

Reported-by: Helge Kreutzmann <debian@helgefjell.de>
Cc: Mario Blaettermann <mario.blaettermann@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

session-keyring.7: SEE ALSO: refer to PAM(7)

Reported-by: Helge Kreutzmann <debian@helgefjell.de>
Cc: Mario Blaettermann <mario.blaettermann@gmail.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

nfsservctl.2: ffix

Reported-by: Helge Kreutzmann <debian@helgefjell.de>
Cc: mario.blaettermann@gmail.com
Signed-off-by: Alejandro Colomar <alx@kernel.org>

socket.7: tfix

The .TP macro is followed by exactly one line of definition, and then
multiple lines of flowed explanatory text.  When flowed, the lack of
periods is apparent in sentences.

Signed-off-by: Jan Engelhardt <jengelh@inai.de>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

getnameinfo.3: SYNOPSIS: Add missing _Nullable qualifiers

Signed-off-by: Alejandro Colomar <alx@kernel.org>

roundup.3: New page documenting roundup(3)

Cc: Paul Eggert <eggert@cs.ucla.edu>
Cc: Wilco Dijkstra <Wilco.Dijkstra@arm.com>
Cc: Adhemerval Zanella <adhemerval.zanella@linaro.org>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

units.7: Remove references to NIST

The SI brochure is the reference.

Signed-off-by: Alejandro Colomar <alx@kernel.org>