Re: Additional Grammar Fixes

[Jason McIntyre]

On Sat, Oct 24, 2020 at 09:46:35AM +, Varik Valefor wrote:
> Sir or Madam:
> 
> Some additional grammatical fixes should be attached to this message.
> 

hi.

i'm afraid most of these are just not fixes. my advice is to send small,
similarly grouped diffs, until you get a feel for what kind of fixes
might get accepted. that would also make it easier to give you feedback
on individual questions.

from your diff i committed the diff below, and have rejected everything
else. i may have missed something, but the diff was so large it was
difficult to read.

thanks,
jmc

Index: lib/libskey/skey.5
===
RCS file: /cvs/src/lib/libskey/skey.5,v
retrieving revision 1.8
diff -u -p -r1.8 skey.5
--- lib/libskey/skey.5  25 Jan 2019 00:19:26 -  1.8
+++ lib/libskey/skey.5  24 Oct 2020 10:30:26 -
@@ -31,7 +31,7 @@ directory contains user records for the 
 system.
 .Pp
 Records take the form of files within
-.Pa /etc/skey
+.Pa /etc/skey ,
 where each file is named for the user whose record it contains.
 For example,
 .Pa /etc/skey/root
@@ -51,7 +51,7 @@ Each record consists of five lines:
 The name of the user the record describes.
 This should be the same as the name of the file.
 .It
-The hash type used for this entry;
+The hash type used for this entry:
 one of md5, sha1, or rmd160.
 The default is md5.
 .It
@@ -62,7 +62,7 @@ Each time the user authenticates via S/K
 A seed used along with the sequence number and the six S/Key words to
 compute the value.
 .It
-The value expected from the crunching of the user's seed, sequence number
+The value expected from the crunching of the user's seed, sequence number,
 and the six S/Key words.
 When the result matches this value, authentication is considered to have
 been successful.
Index: lib/libusbhid/usbhid.3
===
RCS file: /cvs/src/lib/libusbhid/usbhid.3,v
retrieving revision 1.20
diff -u -p -r1.20 usbhid.3
--- lib/libusbhid/usbhid.3  12 May 2020 13:03:52 -  1.20
+++ lib/libusbhid/usbhid.3  24 Oct 2020 10:30:26 -
@@ -86,7 +86,7 @@ The
 .Nm
 library provides routines to extract data from USB Human Interface Devices.
 .Ss INTRODUCTION
-USB HID devices send and receive data laid out in a device dependent way.
+USB HIDs send and receive data laid out in a device dependent way.
 The
 .Nm
 library contains routines to extract the
@@ -117,10 +117,10 @@ return
 .Fa NULL
 on failure.
 .Ss DESCRIPTOR PARSING FUNCTIONS
-To parse the report descriptor the
+To parse the report descriptor, the
 .Fn hid_start_parse
 function should be called with a report descriptor and a set that
-describes which items that are interesting.
+describes which items are interesting.
 The set is obtained by or-ing together values
 .Fa (1 << k)
 where
@@ -198,8 +198,8 @@ A return value of \-1 indicates that an 
 .Va errno
 is set.
 .Ss DATA EXTRACTION FUNCTIONS
-Given the data obtained from a HID device and an item in the
-report descriptor the
+Given the data obtained from a HID and an item in the
+report descriptor, the
 .Fn hid_get_data
 function extracts the value of the item.
 Conversely

Additional Grammar Fixes

[Varik Valefor]

Sir or Madam:

Some additional grammatical fixes should be attached to this message.

KUTGW,
Varik "NOT A COMPUTER PROGRAMMER!!!" Valefor

diff --git a/bin/csh/USD.doc/csh.2 b/bin/csh/USD.doc/csh.2
index af767ea3066..4d977df0529 100644
--- a/bin/csh/USD.doc/csh.2
+++ b/bin/csh/USD.doc/csh.2
@@ -293,7 +293,7 @@ date >!  now
 .DE
 if you really wanted to overwrite the contents of `now'.
 The `>!' is a special metasyntax indicating that clobbering the
-file is ok.\(dg
+file is OK.\(dg
 .FS
 \(dgThe space between the `!' and the word `now' is critical here, as `!now'
 would be an invocation of the
@@ -400,7 +400,7 @@ list, denoting the argument list `\!*'.
 Finally we ran the program `bug' to see that its output is indeed correct.
 .PP
 To make a numbered listing of the program we ran the `cat' command on the file 
`bug.c', with the `-n' flag.
-Unfortunately, we mispelled `cat' as `cay'.
+Unfortunately, we misspelled `cat' as `cay'.  \"hunspell saves the day!
 To correct this
 we used a shell substitute, placing the old text and new text between `^'
 characters.  This is similar to the substitute command in the editor.
diff --git a/bin/stty/stty.1 b/bin/stty/stty.1
index 9faa65de2d5..975a6011757 100644
--- a/bin/stty/stty.1
+++ b/bin/stty/stty.1
@@ -52,7 +52,7 @@ characteristics for the device that is its standard input.
 If no options or operands are specified, it reports the settings of a subset
 of characteristics as well as additional ones if they differ from their
 default values.
-Otherwise it modifies
+Otherwise, it modifies
 the terminal state according to the specified arguments.
 Some combinations of arguments are mutually
 exclusive on some terminal types.
@@ -99,7 +99,9 @@ This corresponds to the
 in the termios structure.
 .Bl -tag -width Ds
 .It Cm parenb Pq Fl parenb
-Enable (disable) parity generation
+Enable
+.Pq disable
+parity generation
 and detection.
 .It Cm parodd Pq Fl parodd
 Select odd (even) parity.
@@ -136,7 +138,8 @@ to
 .Ar number .
 .It Cm hupcl Pq Fl hupcl
 Stop asserting modem control
-(do not stop asserting modem control) on last close.
+.Pq do not stop asserting modem control
+on last close.
 .It Cm hup Pq Fl hup
 Same as
 .Cm hupcl
@@ -186,7 +189,8 @@ Ignore (do not ignore)
 .Dv CR
 on input.
 .It Cm icrnl Pq Fl icrnl
-Map (do not map)
+Map
+.Pq do not map
 .Dv CR
 to
 .Dv NL
@@ -195,7 +199,8 @@ on input.
 Translate (do not translate) upper case to lower case
 on input.
 .It Cm ixon Pq Fl ixon
-Enable (disable)
+Enable
+.Pq disable
 .Dv START/STOP
 output
 control.
@@ -360,8 +365,8 @@ erased with simply an
 .Dv ERASE
 character).
 .It Cm mdmbuf Pq Fl mdmbuf
-If set, flow control output based on condition of Carrier Detect.
-Otherwise
+If set, flow control output is based on the condition of Carrier Detect.
+Otherwise,
 writes return an error if Carrier Detect is low (and Carrier is not being
 ignored with the
 .Dv CLOCAL
diff --git a/lib/libc/gen/getpwnam.3 b/lib/libc/gen/getpwnam.3
index 1936d38026d..6255481c633 100644
--- a/lib/libc/gen/getpwnam.3
+++ b/lib/libc/gen/getpwnam.3
@@ -123,7 +123,7 @@ These file descriptors can be closed by a call to
 It is dangerous for long-running programs to keep the file descriptors
 open as the database will become out of date if it is updated while the
 program is running.
-However the file descriptors are automatically closed when
+However, the file descriptors are automatically closed when
 .Xr execve 2
 is called.
 .Pp
diff --git a/lib/libc/sys/mkdir.2 b/lib/libc/sys/mkdir.2
index f080bacf051..906402dd614 100644
--- a/lib/libc/sys/mkdir.2
+++ b/lib/libc/sys/mkdir.2
@@ -65,9 +65,9 @@ function is equivalent to
 except that where
 .Fa path
 specifies a relative path,
-the newly created directory is created relative to
+the new directory is created relative to
 the directory associated with file descriptor
-.Fa fd
+.Fa fd ,
 instead of the current working directory.
 .Pp
 If
@@ -78,7 +78,7 @@ is passed the special value
 .In fcntl.h )
 in the
 .Fa fd
-parameter, the current working directory is used
+parameter, the current working directory is used,
 and the behavior is identical to a call to
 .Fn mkdir .
 .Sh RETURN VALUES
@@ -87,7 +87,7 @@ and the behavior is identical to a call to
 .Fn mkdir
 and
 .Fn mkdirat
-will fail and no directory will be created if:
+will fail, and no directory will be created if:
 .Bl -tag -width Er
 .It Bq Er ENOTDIR
 A component of the path prefix is not a directory.
@@ -118,7 +118,7 @@ There are no free inodes on the file system on which the
 directory is being created.
 .It Bq Er EDQUOT
 The new directory cannot be created because the user's
-quota of disk blocks on the file system that will
+quota of disk blocks on the file system that would
 contain the directory has been exhausted.
 .It Bq Er EDQUOT
 The user's quota of inodes on the file system on
@@ -189,7 +189,7 @@ Since
 .Fn mkdir
 reappeared in
 .Bx 4.1c ,
-it no longer requires superuser privileges and it automatically creates the
+it no longer