[hackers] [st][PATCH v2] Add support for DSR response "OK" escape sequence

2023-02-08 Thread Adam Price
--- Feedback rightfully indicated I should use strlen() rather than sizeof() to avoid writing an unintentional null byte to the tty. st.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/st.c b/st.c index 34c27ad..603cf95 100644 --- a/st.c +++ b/st.c @@ -1769,11 +1

Re: [hackers] [st][PATCH] Add support for DSR response "OK" escape sequence

2023-02-07 Thread Adam Price
On Tue, Feb 7, 2023 at 2:39 PM Hiltjo Posthuma wrote: > > On Tue, Feb 07, 2023 at 11:20:36AM -0500, Adam Price wrote: > > On Tue, Feb 7, 2023 at 11:17 AM Roberto E. Vargas Caballero > > wrote: > > > > > > On Tue, Feb 07, 2023 at 10:54:57AM -0500, Adam Price w

Re: [hackers] [st][PATCH] Add support for DSR response "OK" escape sequence

2023-02-07 Thread Adam Price
Thanks all! -- Adam

Re: [hackers] [st][PATCH] Add support for DSR response "OK" escape sequence

2023-02-07 Thread Adam Price
On Tue, Feb 7, 2023 at 11:17 AM Roberto E. Vargas Caballero wrote: > > On Tue, Feb 07, 2023 at 10:54:57AM -0500, Adam Price wrote: > > On Mon, Feb 6, 2023 at 12:06 PM Roberto E. Vargas Caballero > > wrote: > > > > > > Hi, > > > > > > On Mon,

Re: [hackers] [st][PATCH] Add support for DSR response "OK" escape sequence

2023-02-07 Thread Adam Price
On Mon, Feb 6, 2023 at 12:06 PM Roberto E. Vargas Caballero wrote: > > Hi, > > On Mon, Feb 06, 2023 at 08:45:27AM +0200, Santtu Lakkala wrote: > > > tsetattr(csiescseq.arg, csiescseq.narg); > > > break; > > > - case 'n': /* DSR – Device Status Report (cursor position) */

[hackers] [st][PATCH] Add support for DSR response "OK" escape sequence

2023-02-05 Thread Adam Price
--- VT100 defines an escape sequence [1] called Device Status Report (DSR). When the DSR sequence received is `csi 5n`, an "OK" response `csi 0n` is returned. This patch adds that "OK" response. I encountered this missing sequence when I noticed that fzf [2] would clobber my prompt whenever comple