On Wed, Aug 16, 2023 at 08:29:23PM -0700, patrick keshishian wrote: > On Wed, Aug 16, 2023 at 7:02???PM Klemens Nanni <k...@openbsd.org> wrote: > > > > Consistent with other options and our manuals in general. > > > > Index: cksum.1 > > =================================================================== > > RCS file: /cvs/src/bin/md5/cksum.1,v > > retrieving revision 1.40 > > diff -u -p -r1.40 cksum.1 > > --- cksum.1 4 Aug 2022 06:20:24 -0000 1.40 > > +++ cksum.1 17 Aug 2023 01:31:19 -0000 > > @@ -131,7 +131,7 @@ Place the checksum into > > .Ar hashfile > > instead of stdout. > > .It Fl p > > -Echoes stdin to stdout and appends the > > +Echo stdin to stdout and append the > > checksum to stdout. > > That sounds awkward. I'd rephrase it as "Echo stdin and append the > checksum to stdout" > Same below with md5.1 instance. >
well, that would be a bit less clear about where stdin goes. i'm not sure it's an improvement. kn: i'm ok with the changes. there's no need to insist on imperative though: i know a lot of people will read it like "(the -p option) echoes stdin" and imperative can sound stiff. still, it's better overall to keep the page consistent, so ok. jmc > --patrick > > > > .It Fl q > > Only print the checksum (quiet mode) or if used in conjunction with the > > @@ -141,16 +141,16 @@ flag, only print the failed cases. > > Reverse the format of the hash algorithm output, making > > it match the checksum output format. > > .It Fl s Ar string > > -Prints a checksum of the given > > +Print a checksum of the given > > .Ar string . > > .It Fl t > > -Runs a built-in time trial. > > +Run a built-in time trial. > > Specifying > > .Fl t > > multiple times results in the number of rounds being multiplied > > by 10 for each additional flag. > > .It Fl x > > -Runs a built-in test script. > > +Run a built-in test script. > > .El > > .Pp > > The default CRC used is based on the polynomial used for CRC error checking > > Index: md5.1 > > =================================================================== > > RCS file: /cvs/src/bin/md5/md5.1,v > > retrieving revision 1.48 > > diff -u -p -r1.48 md5.1 > > --- md5.1 25 Jan 2019 00:19:25 -0000 1.48 > > +++ md5.1 17 Aug 2023 01:31:54 -0000 > > @@ -89,7 +89,7 @@ Place the checksum into > > .Ar hashfile > > instead of stdout. > > .It Fl p > > -Echoes stdin to stdout and appends the > > +Echo stdin to stdout and append the > > checksum to stdout. > > .It Fl q > > Only print the checksum (quiet mode) or if used in conjunction with the > > @@ -100,16 +100,16 @@ Reverse the format of the hash algorithm > > it match the output format used by > > .Xr cksum 1 . > > .It Fl s Ar string > > -Prints a checksum of the given > > +Print a checksum of the given > > .Ar string . > > .It Fl t > > -Runs a built-in time trial. > > +Run a built-in time trial. > > Specifying > > .Fl t > > multiple times results in the number of rounds being multiplied > > by 10 for each additional flag. > > .It Fl x > > -Runs a built-in test script. > > +Run a built-in test script. > > .El > > .Sh EXIT STATUS > > These utilities exit 0 on success, > > >
