[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: Undo for "vi"
From:       Laszlo Papp <lpapp () kde ! org>
Date:       2014-03-19 4:14:45
Message-ID: CAOMwXhNCd3X20K5Q=tFKqGiurKgPD08xMoquCuKr9cJzuS9-mw () mail ! gmail ! com
[Download RAW message or body]

On Tue, Mar 18, 2014 at 5:41 PM, Jody Bruchon <jody@jodybruchon.com> wrote:
> On 3/18/2014 10:51 AM, Jody Bruchon wrote:
>>
>> Out of an interest in seeing this feature, I'm looking at vi.c
>
>
> I've started implementing the "undo" function; it looks to be easier than I
> expected now that I have a feel for the way the buffer is handled and should
> add very little actual code (though obviously it will increase dynamic
> memory usage due to storing undo data somewhere.) If I manage to produce a
> working undo command, I'll send in a patch for Should I make the number of
> undo levels a tunable option via Kconfig? Perhaps via an unlimited option
> (just keep allocating undo data until OOM) and a limited one that allows a
> maximum number of undo levels to be set?

In my opinion, a starting point of X hard coded undo levels is fine
and see how it goes. I personally think it is not worth giving too
much customization right from the beginning.

Thanks for picking this up and working on this!
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread]