[Toybox] [PATCH] vi: added backspace

enh enh at google.com
Wed Feb 1 13:18:51 PST 2023


heh, to be clear: i wasn't "dissing" drive-by patching. it accounts
for at least 80% of my entire career :-)

i use the term just to acknowledge that for some things -- like this
-- there isn't anyone else who's actually working on the thing full
time, which is my personal rationale for wanting "the simplest thing
that could possibly work", and why my definition of "simple" is
something like "the most easily understood by an average programmer
who hasn't seen this particular code before".

(and, yes, in addition to the open() error -- which at least led to a
small simplification of the code -- i've shot myself in the foot by
forgetting that there even are vi tests, not running them, and
breaking them with my recent commit, which i'll have to do something
about before i can sync to AOSP. i'll admit i'm very tempted to just
locally ignore those tests for now so i can get people kicking the
`tar --sort=name` tires asap, and coming back to worry about vi
later!)

On Wed, Feb 1, 2023 at 12:52 PM Jarno Mäkipää <jmakip87 at gmail.com> wrote:
>
> Drive by patching :)
>
> Added better backspace support, now cuts in both insert and normal
> mode and also original text and not just what is in temporary buffer.
> Fix arrow key behavior on insert mode slightly by inserting what ever
> has been typed before moving.
>
> -Jarno
> _______________________________________________
> Toybox mailing list
> Toybox at lists.landley.net
> http://lists.landley.net/listinfo.cgi/toybox-landley.net


More information about the Toybox mailing list