Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm
Precedence: bulk
MIME-Version: 1.0
In-Reply-To: <160112180141.ZM9984@torch.brasslantern.com>
References: <CAKc7PVCnEX1rEhNqhtx7spyUVKCWukz4Tzkqr27-OV9+4hVciQ@mail.gmail.com>
 <CAKc7PVDVR=B8SWX+gSiNvtODWZ-25LtDz7jXVDH8BtrcxMnjXQ@mail.gmail.com>
 <160111161501.ZM5305@torch.brasslantern.com> <CAKc7PVBCEM=46YmgmhPR==Rq+pSfY+hgr8bcOMLV=zLFm6THBg@mail.gmail.com>
 <160112113928.ZM9065@torch.brasslantern.com> <20160113010147.GA4699@tarsus.local2>
 <160112180141.ZM9984@torch.brasslantern.com>
From: Sebastian Gniazdowski <sgniazdowski@gmail.com>
Date: Wed, 13 Jan 2016 17:40:55 +0100
Message-ID: <CAKc7PVBz1nNueRmEfyz3157kvJjEgHW3wmSXraKM+-9P-i5Atw@mail.gmail.com>
Subject: Re: _history-complete-older problems with $(
To: Bart Schaefer <schaefer@brasslantern.com>
Cc: Zsh hackers list <zsh-workers@zsh.org>
Content-Type: multipart/mixed; boundary=001a114024a0aedd16052939d6cb

--001a114024a0aedd16052939d6cb
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On 13 January 2016 at 03:01, Bart Schaefer <schaefer@brasslantern.com> wrot=
e:
> True.  Anyone who wants to fix these issues and post an update is more
> than welcome.  I already spent way more time on it than I intended.

Thanks for the explanations and pointing the manual, this allowed me
to move onward with the code. I didn't use $LBUFFER, but say manually
detected active shell word, and point of break to left and right part
in that word. This will allow to nicely handle COMPLETE_IN_WORD. I
must say it's quite exhausting to handle which word is active. Setting
cursor to "|" in terminal application clarifies few things. Block
cursor highlights a letter, but $CURSOR points before it, where "|"
cursor would stand. Zsh doesn't make following word active, i.e. "ls
|/Users/user_na" with the cursor where "|" is will not complete the
"user_name". I have chosen different approach, in such case I treat
what's after "|" as right part of current shell word. The same I do
for zew-transpose-shell-words (former transpose-segments), block
cursor on first letter of word makes the word active, selected for
transposition.

I do other tricks in the code, e.g. trim left "$((   " out of spaces,
because in case of syntax error (Z+n+) will put spaces in returned
shell word. All this is quite hackish I think. But something could
clarify out of this, and most of all robust Alt-/ is on the horizon.

I have one problem. First compadd returns $found array
populated with matches:

A CURRENT: 3, words: >git,add,wid<
B CURRENT: 3, words: >git,add,wid<
C CURRENT: 3, CURSR: 11, lft: |wid|, rght: ||, words: >git,add,wid<
PREFIX: |wid|, SUFFIX: ||
Calling _history
widen_for_history widen_for_history widen_for_history
widen_for_history widen_for_history widen_for_history
widen_for_history widen_for_history

But nothing is displayed below the prompt, "en" replaces "wid" at
prompt and that's all, while $(<TAB> does:

A CURRENT: 1, words: ><
B CURRENT: 1, words: >$( <
C CURRENT: 1, CURSR: 2, lft: |$(|, rght: ||, words: >$(<
PREFIX: |$(|, SUFFIX: ||
Calling _history
$(( 0 + 1 )) $(( 0 + 1 )) $(( =E8=A9=A6=E5=8F=A5=E7=9B=B8=E7=95=B6=E9=95=B7=
 ""  $(( 0 + 1 )) $( ( echo b ))
$(( echo b ) ) $(( echo a ) ) $(( 0 + 1 )) $(( 0 + 1 )) $(( i + 1 ))
$(( i + 1 )) $(( i + 1 )) $(( i+1 )) $(( i+1 )) $(( i+1 )) $(( i+1 ))
$(( i+1 ))

And below the prompt it's shown:
 ( echo b ))      ( 0 + 1 ))        ( echo a ) )      ( echo b ) )
 ( i + 1 ))        ( i+1 ))          ( =E8=A9=A6=E5=8F=A5=E7=9B=B8=E7=95=B6=
=E9=95=B7 ""

The same problem is with "ls" about which I wrote to you earlier. So
it seems that the second compadd isn't fully working.

The code is here (and also attached):

https://raw.githubusercontent.com/psprint/zsh-editing-workbench/657d6a591be=
a5d26fbed88176ac7de80e8d73927/widen_for_history

Best regards,
Sebastian Gniazdowski

--001a114024a0aedd16052939d6cb
Content-Type: application/octet-stream; name=widen_for_history
Content-Disposition: attachment; filename=widen_for_history
Content-Transfer-Encoding: base64
X-Attachment-Id: f_ijd1yl6o0

ZWNobyAtZSAiXG5BIENVUlJFTlQ6ICRDVVJSRU5ULCB3b3JkczogPiR7KGo6LDopd29yZHN9PCIg
Pj4gL3RtcC93Zmhpc3RvcnkKCndvcmRzPSIkeyhaK24rKUJVRkZFUn0iCmludGVnZXIgbndvcmRz
PSIkeyN3b3Jkc30iCmxvY2FsIGJ1Zj0iJEJVRkZFUiIKCmVjaG8gLWUgIkIgQ1VSUkVOVDogJENV
UlJFTlQsIHdvcmRzOiA+JHsoajosOil3b3Jkc308IiA+PiAvdG1wL3dmaGlzdG9yeQoKIyBSZW1v
dmUgd29yZHMgb25lIGJ5IG9uZSwgY291bnRpbmcgY2hhcmFjdGVycywKIyBjb21wdXRpbmcgYmVn
aW5uaW5nIG9mIGVhY2ggd29yZCwgdG8gZmluZAojIHBsYWNlIHRvIGJyZWFrIHRoZSB3b3JkIGlu
dG8gMiBoYWx2ZXMgKGZvcgojIGNvbXBsZXRlX2luX3dvcmQgb3B0aW9uKQojIElmIHdvdWxkIHVz
ZSAkTEJVRkZFUiwgdGhlbiB3b3VsZCBoYXZlIHRvIGJlIGFibGUKIyB0byBwYXJzZSBzaGVsbCB3
b3JkcywgdG8gZXh0cmFjdCB3b3JkJ3MgaGFsdmVzCgpsb2NhbCBpIHdvcmQKaW50ZWdlciBjaGFy
X2NvdW50PTAKdHlwZXNldCAtYSB3b3JkX2JlZ2lubmluZ3MKQ1VSUkVOVD0iLTEiCgpmb3IgKCgg
aT0xOyBpPD1ud29yZHM7IGkrKyApKTsgZG8KICAgICMgUmVtb3ZlIHRyYWlsaW5nIHNwYWNlcyBm
cm9tICR3b3JkIC0gdGhleSBtYXkKICAgICMgYXBwZWFyIGluIGNhc2Ugb2Ygc3ludGF4IGVycm9y
LCBsaWtlICIkKCggICA8VEFCPiIKICAgIHdvcmRzW2ldPSIke3dvcmRzW2ldJSUgIyN9IgoKICAg
IHdvcmQ9IiR7d29yZHNbaV19IgoKICAgICMgUmVtb3ZlIHdoaXRlIHNwYWNlcwogICAgYnVmPSIk
e2J1ZiMjKCNtKVteJHdvcmRbMV1dI30iCiAgICAjIENvdW50IHRoZW0KICAgIGNoYXJfY291bnQ9
Y2hhcl9jb3VudCsiJCNNQVRDSCIKICAgICMgVGhpcyBpcyB0aGUgYmVnaW5uaW5nIG9mIGN1cnJl
bnQgd29yZAogICAgd29yZF9iZWdpbm5pbmdzW2ldPSIkY2hhcl9jb3VudCIKCiAgICAjIFJlbW92
ZSB0aGUgd29yZAogICAgTUFUQ0g9IiIKICAgIGJ1Zj0iJHtidWYjKCNtKSR3b3JkfSIKCiAgICAj
IElmIHNoZWxsIHdvcmQgbm90IGZvdW5kLCByZXR1cm4uIFRoaXMgc2hvdWRsbid0IGhhcHBlbgog
ICAgWyAteiAiJE1BVENIIiBdICYmIHsgZWNobyAiQWJvcnRpbmciID4+IC90bXAvd2ZoaXN0b3J5
OyByZXR1cm4gMCB9CgogICAgIyBTcGFjZXMgcG9pbnQgdG8gcHJldmlvdXMgc2hlbGwgd29yZAog
ICAgW1sgIiRDVVJSRU5UIiAtZXEgIi0xIiAmJiAiJGNoYXJfY291bnQiIC1ndCAiJENVUlNPUiIg
XV0gJiYgQ1VSUkVOVD1pLTEKCiAgICAjIEFjdHVhbCBjaGFyYWN0ZXJzIHBvaW50IHRvIGN1cnJl
bnQgc2hlbGwgd29yZAogICAgY2hhcl9jb3VudD1jaGFyX2NvdW50KyIkI3dvcmQiCiAgICBbWyAi
JENVUlJFTlQiIC1lcSAiLTEiICYmICIkY2hhcl9jb3VudCIgLWdlICIkQ1VSU09SIiBdXSAmJiBD
VVJSRU5UPWkKZG9uZSAKCiMgV2hhdCdzIGxlZnQgaW4gJGJ1ZiBjYW4gYmUgb25seSB3aGl0ZSBz
cGFjZXMKY2hhcl9jb3VudD1jaGFyX2NvdW50KyIkI2J1ZiIKW1sgIiRDVVJSRU5UIiAtZXEgLTEg
JiYgIiRjaGFyX2NvdW50IiAtZ2UgIiRDVVJTT1IiIF1dICYmIENVUlJFTlQ9aS0xCgojIERpdmlk
ZSB3b3Jkc1tDVVJSRU5UXSBpbnRvIHR3byBoYWx2ZXMKaW50ZWdlciBkaWZmPSQoKCBDVVJTT1Ig
LSB3b3JkX2JlZ2lubmluZ3NbQ1VSUkVOVF0gKSkKd29yZD0iJHt3b3Jkc1tDVVJSRU5UXX0iCmxv
Y2FsIGxlZnQ9IiR7d29yZFsxLGRpZmZdfSIKbG9jYWwgcmlnaHQ9IiR7d29yZFtkaWZmKzEsLTFd
fSIKCmVjaG8gIkMgQ1VSUkVOVDogJENVUlJFTlQsIENVUlNSOiAkQ1VSU09SLCBsZnQ6IHwkbGVm
dFsqXXwsIHJnaHQ6IHwkcmlnaHR8LCB3b3JkczogPiR7KGo6LDopd29yZHN9PCIgPj4gL3RtcC93
Zmhpc3RvcnkKCiMgSGlzdG9yeSBpcyBzZWFyY2hlZCBmb3Igd29yZHMgdGhhdCBzdGFydCB3aXRo
ICRQUkVGSVgKUFJFRklYPSIke2xlZnR9IgpJUFJFRklYPScnCiMgLi4uIGFuZCBlbmQgd2l0aCAk
U1VGRklYClNVRkZJWD0iJHtyaWdodH0iCklTVUZGSVg9JycKCmVjaG8gIlBSRUZJWDogfCRQUkVG
SVh8LCBTVUZGSVg6IHwkU1VGRklYfCIgPj4gL3RtcC93Zmhpc3RvcnkKCnsKICAgIGNvbXBhZGQg
KCkgewogICAgICAgIGxvY2FsIC1hIGZvdW5kCiAgICAgICAgYnVpbHRpbiBjb21wYWRkIC1PIGZv
dW5kICIkQCIKICAgICAgICBbWyAiJCNmb3VuZCIgLWVxIDAgXV0gJiYgZWNobyAtbiAiX2VtcHR5
XyAiID4+IC90bXAvd2ZoaXN0b3J5IHx8IGVjaG8gIiR7Zm91bmRbQF19IiA+PiAvdG1wL3dmaGlz
dG9yeQogICAgICAgIGJ1aWx0aW4gY29tcGFkZCAtUSAtVSAiJHsoQCkkeyhAKWZvdW5kIyRQUkVG
SVh9JSRTVUZGSVh9IgogICAgfQogICAgZWNobyAiQ2FsbGluZyBfaGlzdG9yeSIgPj4gL3RtcC93
Zmhpc3RvcnkKICAgIF9oaXN0b3J5CiAgICBlY2hvID4+IC90bXAvd2ZoaXN0b3J5Cn0gYWx3YXlz
IHsKICAgIHVuZnVuY3Rpb24gY29tcGFkZAp9CgojIHZpbTpmdD16c2gK
--001a114024a0aedd16052939d6cb--