From bc053e4879958b53a0db089282c8a044757e5754 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Daniel=20Ekl=C3=B6f?= <daniel@ekloef.se>
Date: Fri, 15 Jan 2021 18:40:07 +0100
Subject: [PATCH] vt: document correct BS behavior, and why we do differently

---
 vt.c | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/vt.c b/vt.c
index e8e10b2f..bbb7e990 100644
--- a/vt.c
+++ b/vt.c
@@ -129,10 +129,22 @@ action_execute(struct terminal *term, uint8_t c)
 
     case '\b':
         /* backspace */
+#if 0
+        /*
+         * This is the “correct” BS behavior. However, it doesn’t play
+         * nicely with bw/auto_left_margin, hence the alternative
+         * implementation below.
+         *
+         * Note that it breaks vttest “1. Test of cursor movements ->
+         * Test of autowrap”
+         */
+        term_cursor_left(term, 1);
+#else
         if (term->grid->cursor.lcf)
             term->grid->cursor.lcf = false;
         else
             term_cursor_left(term, 1);
+#endif
         break;
 
     case '\t': {