cminyard/gensio

man syntax warnings

Zugschlus opened this issue · 1 comments

Hi,

checking the man page for man/gensio_set_sync.3 gives a warning:
LC_ALL=en_US.UTF-8 MANROFFSEQ='' MANWIDTH=80 man --warnings -E UTF-8 -l -Tutf8 -Z man/gensio_set_sync.3 >/dev/null troff: :55: warning: macro '..' not defined

I guess this is missing leading whitespace, as this patch here:

--- gensio_set_sync.3   2020-09-26 17:57:52.324897967 +0200
+++ man/gensio_set_sync.3       2020-09-26 16:24:39.344583049 +0200
@@ -52,7 +52,7 @@
    call fd1_write_handler()
 .br
 if (fd2 read is set)
-   ...
+...
 .PP
 The gensio handling does all this for you.  Just register a handler
 with the gensio to get the read and write calls.  It's more efficient,

Fixes the warning and the output looks more as it was intended.

Similiar issues can be fixed by applying the following trivial patches:

--- a/man/gensio.5
+++ b/man/gensio.5
@@ -123,7 +123,7 @@ Some gensio types support serial port se
 serial ports, IPMI Serial Over LAN, and telnet with RFC2217 enabled.
 
 A client serial gensio can set and get serial port options using the
-.Bsergensio_xxx()
+.B sergensio_xxx()
 functions.  Server serial gensios receive requests from the client via
 .I GENSIO_EVENT_SER_xxx
 events in the callback.
@@ -517,7 +517,7 @@ A file will be created with the given so
 permissions to create a writeable file in that location.  If the file
 already exists, an error will be returned on an accepter socket unless
 you specify
-.i delsock
+.I delsock
 which will cause the file to be deleted.
 
 You should read the unix(7) man page for details on the semantics of

Ok, these are fixed. The end of your post was cut off, not sure what you were trying to say. Thanks for the fixes.