Re: Invisible PROMPT2

On Mon, Dec 23, 2019 at 5:43 AM Maxence Ahlouche
<maxence(dot)ahlouche(at)gmail(dot)com> wrote:
> On Wed, 27 Nov 2019 at 17:09, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Good idea, but I think you need to account for "visible" (ie, if the
>> newline is inside RL_PROMPT_START_IGNORE, it shouldn't change the width).
>> It might be best to add logic inside the existing "if (visible)" instead
>> of making a new top-level case.
>
> Right, I assumed that it was safe given that only terminal control characters were invisible.
> Since the title of the terminal window can be changed as well via control characters, it's probably better not to make that assumption.
>
> I updated the patch accordingly.

Pushed. Thanks!