Ok, here comes a second attempt. It works with your test page and avoids the memore leak. Please test. Cheers, Johannes On Tue, May 20, 2008 at 05:50:35PM -0400, Jorge Arellano Cid wrote:
On Tue, May 20, 2008 at 07:16:45PM +0200, Johannes Hofmann wrote:
Hi,
attached is a small patch to improve memory usage. It creates a single word for a sequence of spaces in PRE mode instead of one word per character.
Good idea, but it's not the same rendering. Check the offsets with the attached file.
BTW, there's a memory leak when len=0 (bacause Dstr allocates for the string). While reviewing I thought why not just count the spaces and use dStrnfill(num_spc, ' ').
-- Cheers Jorge.-
<html> <pre> <P>Bla bla bla. Blabla blabla bla blala bla bla. Blabla blabla bla bla. <P> Blala bla. Blabla blabla bla blaba bla bla. Bla blabla bla bla. <BR> Blilly blilly bla bla bla. Blabla blabla bla bla. <HR> Lallyla layllaylay layla layla lally lally. </pre> </html>
_______________________________________________ Dillo-dev mailing list Dillo-dev@dillo.org http://lists.auriga.wearlab.de/cgi-bin/mailman/listinfo/dillo-dev