Re: [PATCH] prompt: support generic non-visible regions

[Roman Perepelitsa <roman.perepelitsa@xxxxxxxxx>]

On Thu, Aug 11, 2022 at 7:24 PM Roman Perepelitsa
<roman.perepelitsa@xxxxxxxxx> wrote:
>
> On Thu, Aug 11, 2022 at 4:37 PM Roman Perepelitsa
> <roman.perepelitsa@xxxxxxxxx> wrote:
> >
> > I like this patch [...]
>
> I should've said that I like the idea behind it. The patch implements
> one part of it. To complete the implementation percent expansion needs
> to output \001 and \002.

I apologize for replying to my own message multiple times.

I just realized that what I suggested may not work. Consider this code:

    print -r -- ${(%):-'%{hello%}'}

Currently it is equivalent to this:

    print -r -- 'hello'

However, if percent expansion starts inserting \001 and \002, the same
code will be equivalent to this:

    print -r -- $'\001hello\002'

This may be undesirable.

The patch by Felipe, as it stands, doesn't have this problem.

Roman.