On Mon, Dec 23, 2019 at 5:43 AM Maxence Ahlouche <maxence.ahlou...@gmail.com> wrote: > On Wed, 27 Nov 2019 at 17:09, Tom Lane <t...@sss.pgh.pa.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!
