Re: lynx-dev lynx2.8.5dev.12d.patch.gz

[Thomas Dickey]

On Thu, Jan 02, 2003 at 01:45:32PM +0300, Leonid Pauzner wrote:
> >>> 1-Jan-2003 18:37 Thomas Dickey wrote:
> >>> > I put a snapshot of my current patch on
> >>> >       ftp://invisible-island.net/temp/
> >>>
> >>> > which includes LP's HTLine change, modified so it seems to work 
> >>> > properly.
> 
> > This variant did not leak (comparing to dev.11) -
> > it allocates *too much memory* in the pool,
> 
> > There may be another palliative variant using freeHTLine:
> > we malloc/free the last line each time, but other lines
> > are allocated in the pool. (To free the last line
> > we store its pointer in other place). May supply a patch.
> 
> Here it is, against your '12d' snapshot

perhaps something like that.  But I just tested this one, and it has two
problems:

        a) refers to freed memory (same as the previous report), and

        b) doesn't repaint the display properly.  Unlike the previous
           case, this time it doesn't repaint the parent properly - half
           the time.

I'm considering moving your fixed HTLine's (tmp_long_line) into the document
structure, and seeing how your original patch does that way.  From debugging, I
can see that the problem with redisplay occurs with the second entry
of that array.

-- 
Thomas E. Dickey <address@hidden>
http://invisible-island.net
ftp://invisible-island.net

; To UNSUBSCRIBE: Send "unsubscribe lynx-dev" to address@hidden