Re: That infinite loop in Geoff's second zle_refresh patch

[Geoff Wing <mason@xxxxxxxxxxxxx>]

pws@xxxxxx wrote:
:schaefer@xxxxxxxxxxxxxxxxxxxxxxx wrote:
:> The patch below changes starting count from 0 to 1, which stops the
:> infinite loop, but feels wrong to me.  I can't see any reason to call
:> tcdelcost(0), but that code was there before either of Geoff's recent
:> patches; the only nearby difference since pre2 is a change from

Looks fine to me.  I give it my blessing :-)  
The "tcdelcost(i) < pfxlen(p1, nl)" stuff is only a rough estimate anyway.
You'd need to add costs of moving right and other stuff which is more
complex than justifies the reward (eg. you might save one or two chars 
output - and there are other areas that could use one or two character
cleanup first).

:This has a bad effect:  inserting or deleting on a continued line
:before the end of the line redisplays the rest of line wrongly.  I
:haven't looked into the details.  (Maybe I got confused; my
:zle_refresh.c is different from the one in the patch, so I'm either
:more or less up to date.)

Um, what stuffs up?  Is this on an xterm?  An example?
I fixed up some of the continued line stuff so cut/paste works better on
xterms and similar 'cos I seem to remember you were one of those asking
for it(?)
-- 
Mason [G.C.W]  mason@xxxxxxxxxxxxxxxxxx    "Hurt...Agony...Pain...LOVE-IT"