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 <[EMAIL PROTECTED]>
http://invisible-island.net
ftp://invisible-island.net
; To UNSUBSCRIBE: Send "unsubscribe lynx-dev" to [EMAIL PROTECTED]
