On Sat, Sep 10, 2022 at 11:46:21PM +0200, Ingo Schwarze wrote: > Hi, > > yes, this is completely correct, with one tiny exception that should > be fixed while committing, see in-line below. > > Jason, since you already started working on this, could you please > commit this patch with OK schwarze@? > > I'm surprised there was still so much .Li in our tree where .Vt > should have been. These are not even edge cases but completely > unambiguous .Vt. > > Note that the mdoc(7) manual deprecates .Li (it is a presentational > macro with an invisible effect - we usually want semantic rather > than presentational markup). Rare cases exist where it may not be > completely obvious what to use instead, but here it is. > > Thanks, > Ingo >
thanks for checking ingo. committed now with the change you mentioned (and zero markup). thanks for the diff, josiah. jmc > > Josiah Frentsos wrote on Sat, Sep 10, 2022 at 12:29:28PM -0400: > > > Index: lib/libc/gen/frexp.3 > > Index: lib/libc/gen/getgrent.3 > > Index: lib/libc/gen/getpwent.3 > > Index: lib/libc/gen/getpwnam.3 > > Index: lib/libc/gen/glob.3 > > Index: lib/libc/gen/isalnum.3 > > Index: lib/libc/gen/isalpha.3 > > Index: lib/libc/gen/isblank.3 > > Index: lib/libc/gen/iscntrl.3 > > Index: lib/libc/gen/isdigit.3 > > Index: lib/libc/gen/isgraph.3 > > Index: lib/libc/gen/islower.3 > > Index: lib/libc/gen/isprint.3 > > Index: lib/libc/gen/ispunct.3 > > Index: lib/libc/gen/isspace.3 > > Index: lib/libc/gen/isupper.3 > > Index: lib/libc/gen/isxdigit.3 > > Index: lib/libc/gen/lockf.3 > > Index: lib/libc/gen/login_cap.3 > > Index: lib/libc/gen/modf.3 > > Index: lib/libc/gen/opendir.3 > > Index: lib/libc/gen/setjmp.3 > > Index: lib/libc/gen/times.3 > > Index: lib/libc/gen/tolower.3 > > Index: lib/libc/gen/toupper.3 > > Index: lib/libc/gen/uname.3 > > Index: lib/libc/gen/utime.3 > > Index: lib/libc/locale/localeconv.3 > > Index: lib/libc/net/ether_aton.3 > > Index: lib/libc/net/getaddrinfo.3 > > Index: lib/libc/net/getnameinfo.3 > > Index: lib/libc/net/getpeereid.3 > > Index: lib/libc/net/getrrsetbyname.3 > > Index: lib/libc/net/htonl.3 > > =================================================================== > > RCS file: /cvs/src/lib/libc/net/htonl.3,v > > retrieving revision 1.5 > > diff -u -p -r1.5 htonl.3 > > --- lib/libc/net/htonl.3 13 Feb 2019 07:02:09 -0000 1.5 > > +++ lib/libc/net/htonl.3 10 Sep 2022 16:10:01 -0000 > > @@ -66,14 +66,14 @@ or > > .Sq l ) > > is a mnemonic > > for the traditional names for such quantities, > > -.Li short > > +.Vt short > > and > > -.Li long , > > +.Vt long , > > respectively. > > This is misleading, as explained in the very next sentence. > I suggest just dropping the .Li markup in these two instances > without any replacement, or .Dq if you insist on some markup. > > > Today, the C concept of > > -.Li short > > +.Vt short > > and > > -.Li long > > +.Vt long > > integers need not coincide with this traditional misunderstanding. > > On machines which have a byte order which is the same as the network > > order, routines are defined as null macros. > > This part is correct. > > > Index: lib/libc/net/inet_addr.3 > > Index: lib/libc/net/inet_net_ntop.3 > > Index: lib/libc/net/inet_ntop.3 > > Index: lib/libc/regex/regex.3 > > Index: lib/libc/rpc/xdr.3 > > Index: lib/libc/stdio/fseek.3 > > Index: lib/libc/stdio/getc.3 > > Index: lib/libc/stdio/putc.3 > > Index: lib/libc/stdio/ungetc.3 > > Index: lib/libc/stdlib/atof.3 > > Index: lib/libc/stdlib/atoi.3 > > Index: lib/libc/stdlib/atol.3 > > Index: lib/libc/stdlib/atoll.3 > > Index: lib/libc/stdlib/div.3 > > Index: lib/libc/stdlib/getopt_long.3 > > Index: lib/libc/stdlib/imaxdiv.3 > > Index: lib/libc/stdlib/ldiv.3 > > Index: lib/libc/stdlib/lldiv.3 > > Index: lib/libc/stdlib/strtod.3 > > Index: lib/libc/stdlib/strtonum.3 > > Index: lib/libc/string/memccpy.3 > > Index: lib/libc/string/memchr.3 > > Index: lib/libc/string/memcmp.3 > > Index: lib/libc/string/memset.3 > > Index: lib/libc/sys/accept.2 > > Index: lib/libc/sys/fcntl.2 > > Index: lib/libc/sys/getpeername.2 > > Index: lib/libc/sys/getrlimit.2 > > Index: lib/libc/sys/getsockname.2 > > Index: lib/libc/sys/getsockopt.2 > > Index: lib/libc/sys/ioctl.2 > > Index: lib/libc/sys/ptrace.2 > > Index: lib/libc/sys/quotactl.2 > > Index: lib/libc/termios/tcsetattr.3 > > Index: lib/libc/time/ctime.3 > > Index: lib/libradius/radius_new_request_packet.3 > > Index: share/man/man3/bit_alloc.3 > > Index: share/man/man3/dl_iterate_phdr.3 > > Index: share/man/man4/bpf.4 > > Index: share/man/man4/ddb.4 > > Index: share/man/man4/openprom.4 > > Index: share/man/man4/options.4 > > Index: share/man/man4/speaker.4 > > Index: share/man/man5/ranlib.5 > > Index: share/man/man8/crash.8 > > Index: share/man/man9/printf.9 > > Index: share/man/man9/socreate.9 > > Index: share/man/man9/style.9 > > Index: usr.bin/ssh/sshd.8 > > Index: usr.sbin/zdump/zdump.8
