Re: [PATCH] Restrict named directories to scalar parameters.
Philippe Altherr <[email protected]> Sun, 19 Jul 2026 15:53:24 +0200
| Newsgroups | gmane.comp.shells.zsh.devel |
|---|---|
| Message-ID | <CAGdYchtvRh8HgwBUZRUpyb3=OvQ=c+wqKn62a2fherV3JBafjQ@mail.gmail.com> |
--000000000000ba9d040656f71d2d Content-Type: multipart/alternative; boundary="000000000000ba9d030656f71d2b" --000000000000ba9d030656f71d2b Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Mon, Jun 22, 2026 at 6:03=E2=80=AFAM Bart Schaefer <schaefer@brasslanter= n.com> wrote: > On Tue, Jun 16, 2026 at 1:43=E2=80=AFPM Philippe Altherr > <[email protected]> wrote > > > > Here is an updated patch where the test uses the null PS1 trick. > > > > + echo "nameddirs=3D( ${(kv)nameddirs} )" > > Noting that this is done because nameddirs is a hidden parameter, so > "typeset -p" won't print its value. Worth mentioning in the file? > Indeed, I would have prefered using "typeset -p nameddirs". We could add the following comment. - Add comment about "typeset -p nameddirs" <https://github.com/zsh-users/zsh/compare/master...paltherr:zsh:add-comment= -about-typeset-p-nameddirs> Philippe --000000000000ba9d030656f71d2b Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr"><span style=3D"background-color:transpare= nt">On Mon, Jun 22, 2026 at 6:03=E2=80=AFAM Bart Schaefer <<a href=3D"ma= ilto:[email protected]">[email protected]</a>> wrote:</s= pan></div><div class=3D"gmail_quote gmail_quote_container"><blockquote clas= s=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid r= gb(204,204,204);padding-left:1ex">On Tue, Jun 16, 2026 at 1:43=E2=80=AFPM P= hilippe Altherr<br> <<a href=3D"mailto:[email protected]" target=3D"_blank">philipp= [email protected]</a>> wrote<br> ><br> > Here is an updated patch where the test uses the null PS1 trick.<br> ><br> > +=C2=A0 =C2=A0echo "nameddirs=3D( ${(kv)nameddirs} )"<br> <br> Noting that this is done because nameddirs is a hidden parameter, so<br> "typeset -p" won't print its value.=C2=A0 Worth mentioning in= the file?<br></blockquote><div><br></div><div>Indeed, I would have prefere= d using "typeset -p nameddirs". We could add the following commen= t.</div><div><br></div><div>-=C2=A0<a href=3D"https://github.com/zsh-users/= zsh/compare/master...paltherr:zsh:add-comment-about-typeset-p-nameddirs">Ad= d comment about "typeset -p nameddirs"</a></div><div><br></div><d= iv>Philippe</div><div><br></div></div></div> --000000000000ba9d030656f71d2b-- --000000000000ba9d040656f71d2d Content-Type: text/plain; charset="US-ASCII"; name="add-comment-about-typeset-p-nameddirs.txt" Content-Disposition: attachment; filename="add-comment-about-typeset-p-nameddirs.txt" Content-Transfer-Encoding: base64 Content-ID: <f_mrruskqf0> X-Attachment-Id: f_mrruskqf0 ZGlmZiAtLWdpdCBhL1Rlc3QvSzAxbmFtZXJlZi56dHN0IGIvVGVzdC9LMDFuYW1lcmVmLnp0c3QK aW5kZXggOTUyYmJkMzBjLi45NWI4ZTVkYmMgMTAwNjQ0Ci0tLSBhL1Rlc3QvSzAxbmFtZXJlZi56 dHN0CisrKyBiL1Rlc3QvSzAxbmFtZXJlZi56dHN0CkBAIC0xNTc1LDYgKzE1NzUsNyBAQCBGOnBy ZXZpb3VzbHkgdGhpcyBjb3VsZCBjcmVhdGUgYW4gaW5maW5pdGUgcmVjdXJzaW9uIGFuZCBjcmFz aAogCiAjIFRlc3RpbmcgbmFtZWQgZGlyZWN0b3JpZXMgcmVxdWlyZXMgdG8gcnVuIGluIGludGVy YWN0aXZlIG1vZGUKIAorICMgVXNlICIkeyhrdiluYW1lZGRpcnN9IiBiZWNhdXNlICJ0eXBlc2V0 IC1wIG5hbWVkZGlycyIgcHJpbnRzIG5vIHZhbHVlCiAgUFMxPSAkWlRTVF90ZXN0ZGlyLy4uL1Ny Yy96c2ggLWZpcyA8PDwnCiAgICBbWyAtZCBNb2R1bGVzL3pzaCBdXSAmJiBtb2R1bGVfcGF0aD0o ICRQV0QvTW9kdWxlcyApCiAgICB0eXBlc2V0IHYxPS8xIHYyCg== --000000000000ba9d040656f71d2d--