Thread (21 messages) 21 messages, 4 authors, 2020-09-10

Re: [PATCH v2 1/2] Fit to Plan 9's ANSI/POSIX compatibility layer

From: Kyohei Kadota <hidden>
Date: 2020-09-10 02:46:43

Thank you for your review!

2020年9月10日(木) 4:56 Eric Sunshine [off-list ref]:
On Wed, Sep 9, 2020 at 3:48 PM Kyohei Kadota via GitGitGadget
[off-list ref] wrote:
quoted
And its sed(1)'s label is limited to maximum seven characters.
Therefore I replaced some labels to drop a character.

* close -> cl
* continue -> cont (cnt is used for count)
* line -> ln
* hered -> hdoc
* shell -> sh
* string -> str
These are reasonable. "cl" feels a little odd, but I can't think of
anything better.
quoted
diff --git a/generate-cmdlist.sh b/generate-cmdlist.sh
@@ -24,7 +24,7 @@ category_list () {
 get_synopsis () {
        sed -n '
-               /^NAME/,/'"$1"'/H
+               /^NAME/,/'"$1"'/h
This change is not mentioned in the commit message. "H" and "h" are
very different commands, so it's difficult, at a glance, to tell if
this change is even valid. Some explanation in the commit message
would help (if it is indeed valid).
I missed, this change was not needed. It is remnants of trial and error.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help