Here's some more documentation clean-up for ksh-20140929. First, in
the attached builtin.c.diff are a couple of corrections to the read
built-in. In the description of read's -C option I figure the statement:
... Equivalent to -aksh.
Should actually be:
... Equivalent to -mksh.
It just makes more sense that way. Then, in the description of the -N
option I eliminated the stray "n" in the statement:
Read exactly ncount characters. ... .
So that it now reads:
Read exactly count characters. ... .
Since I had already made changes to src/cmd/ksh93/data/builtin.c, I
decided to add a missing space character I happened to see in the
documentation for the kill built-in. It's minor, but like I said I had
already made changes to builtin.c, so why not?
The attached builtin.c.diff is a cumulative patch that includes the
patch I sent a while back in
http://lists.research.att.com/pipermail/ast-users/2014q4/004691.html.
Second, the documentation for the complete/compgen built-ins is kind
of messy. I think I made the correct changes. Some were obvious. But, I
have to admit I made a couple of outright guesses. The changes are
attached in pcomplete.c.diff. I won't go through all the changes here,
but the patch should probably be scrutinized to make sure the changes I
made are alright.
Terrence Doyle
--- src/cmd/ksh93/data/builtins.c.orig 2014-07-21 10:17:25.000000000 -0500
+++ src/cmd/ksh93/data/builtins.c 2015-01-29 16:28:07.000000000 -0600
@@ -437,7 +437,7 @@
"specified with separate invocations of \bbuiltin\b. Libraries are "
"searched in the reverse order in which they are specified.]"
"[n?Disable each of the specified built-ins. Special built-ins cannot be "
- "disabled. If no built-ns are specifed, display all disabled built-ins.]]"
+ "disabled. If no built-ns are specifed, display all disabled built-ins.]"
"[l?List the library base name, plugin YYYYMMDD version stamp, and full "
"path for \b-f\b\alib\a on one line on the standard output.]"
"[p?Causes the output to be in a form of \bbuiltin\b commands that can be "
@@ -1041,7 +1041,7 @@
"The \b-n\b and \b-s\b options cannot be specified.]"
"[q]#[n?On systems that support \asigqueue\a(2), send a queued signal with "
"message number than can only be as large as a signed integer. The "
- "value is stored in the int portion of the value field."
+ "value is stored in the int portion of the value field. "
"The \ajob\a must be specified as a positive number. "
"On systems that do not support \asigqueue\a(2), a signal "
"is sent without the message number \an\a and will not be queued.]"
@@ -1330,7 +1330,7 @@
"[A?Unset \avar\a and then create an indexed array containing each field in "
"the line starting at index 0.]"
"[C?Unset \avar\a and read \avar\a as a compound variable. Equivalent to "
- "\b-aksh\b]"
+ "\b-mksh\b.]"
"[a?Unset \avar\a and then create an indexed array containing each field in "
"the line starting at index 0.]"
"[d]:[delim?Read until delimiter \adelim\a instead of to the end of line.]"
@@ -1352,7 +1352,7 @@
"a terminal or pipe.]"
"[n]#[count?Read at most \acount\a characters. For binary fields \acount\a "
"is the number of bytes.]"
-"[N]#[count?Read exactly \ancount\a characters. For binary fields \acount\a "
+"[N]#[count?Read exactly \acount\a characters. For binary fields \acount\a "
"is the number of bytes.]"
"[v?When reading from a terminal the value of the first variable is displayed "
"and used as a default value.]"
--- src/cmd/ksh93/edit/pcomplete.c.orig 2014-09-01 12:03:19.000000000 -0500
+++ src/cmd/ksh93/edit/pcomplete.c 2015-01-17 17:50:27.000000000 -0600
@@ -140,7 +140,7 @@
"[+disabled?Names of disabled shell builtins.]"
"[+enabled?Names of enabled shell builtins.]"
"[+export?Equivalent to \b-e\b.]"
- "[+file??Equivalent to \b-f\b.]"
+ "[+file?Equivalent to \b-f\b.]"
"[+function?Names of shell functions.]"
"[+group?Equivalent to \b-g\b.]"
"[+helptopic?Help topics as accepted by the \bhelp\b builtin.]"
@@ -150,7 +150,7 @@
"[+keyword?Equivalent to \b-k\b.]"
"[+running?Names of running jobs if job control is active.]"
"[+service?Equivalent to \b-s\b.]"
- "[+setopt?Valid option names as acceptered by the \bshopt\b builtin.]"
+ "[+setopt?Valid option names as accepted by the \bshopt\b builtin.]"
"[+signal?Signal names.]"
"[+stopped?Names of stopped jobs if job control is active.]"
"[+user?Equivalent to \b-u\b.]"
@@ -158,10 +158,10 @@
"}"
"[o]:[option?\aoption\a may be one of the following list of completions:]{"
"[+bashdefault?perform \bbash\b default completions if there are no "
- "matches]"
- "[+default?Use default filename completion of there are no matches.]"
+ "matches.]"
+ "[+default?Use default filename completion if there are no matches.]"
"[+dirnames?Perform directory name completion if there are no matches.]"
- "[+filenames?]"
+ "[+filenames?Perform filename completion if there are no matches.]"
"[+nospace?No space will be appended to words completed at the end "
"of the line.]"
"}"
@@ -176,16 +176,15 @@
"[D?Use this completion as a default for commands that don't have a completion
"
"specified.]"
"[E?Use this completion when for a blank line.]"
- "specified.]"
"[F]:[function?The shell function \afunction\a is executed in the current "
"environment. When it finishes the possible list of completions are "
- "retrieved from the indexed array variable \bCOMPREPLY\b]"
+ "retrieved from the indexed array variable \bCOMPREPLY\b.]"
"[X]:[pattern?\apattern\a is used to filter filename expansions. It is "
"applied to the list of possible completions generated by the "
"preceding options and arguments, and each completion matching "
"\apattern\a is removed from the list. A leading \b!\b in \apattern\a "
"negates the pattern.]"
-"[P]:[prefix?\aprefix\a is prepended to each possible completion after all "
+"[P]:[prefix?\aprefix\a is prefixed to each possible completion after all "
"other options have been applied.]"
"[S]:[suffix?\asuffix\a is appended to each possible completion after all "
"other options have been applied.]"
_______________________________________________
ast-developers mailing list
ast-developers@lists.research.att.com
http://lists.research.att.com/mailman/listinfo/ast-developers
