May 2009 - Dillo-dev - lists.mailman3.com

[bug]: uint32_t not declared

by onepoint＠starurchin.org

The latest tip fails to build for me. I don't have uint32_t. Which header should include it? I'm using gcc 4.1.2. Regards, Jeremy Henty

May 24, 2009

[patch]: [doc]: escape some hash in the text

by onepoint＠starurchin.org

When you're sick of bugs, clean up the docs! This escapes some '#' characters so that "#define" and "#include" appear properly in the text. It also silences some Doxygen warnings (it was trying to interpret those words as links). Regards, Jeremy Henty

May 23, 2009

Latest on invalid image callback references

by onepoint＠starurchin.org

On Fri, May 22, 2009 at 02:31:58PM -0400, Jorge Arellano Cid wrote: > On Tue, May 19, 2009 at 07:34:30AM +0100, Jeremy Henty wrote: > > > I have just applied a patch I was considering earlier that makes > > a_Cache_stop_client() close the client. And now I have just unapplied it because it crashed Dillo all the time. <sigh> If this patch isn't wrong, then something else is. > A huge step forward would be to find a way to repoduce the bug. Believe me, I know. Unfortunately my current patches do little more than confirm at the end of the session that sometime previously, something, somewhere went wrong. Which is great for confirming the existence of the bug, but for fixing it, not so much. What's really needed is complete logging of the CCC, the cache and the dicache. Back to the drawing board... > My guess is that this semantic is still fuzzy ... It's nice to know that it isn't just me who doesn't understand the code! :-) Just what is a repush anyway? (BTW, I'm not complaining, just pining for the good old days when Dillo was so buggy that it was easy for me to find something I could actually fix. :-) :-) ) Regards, Jeremy Henty PS. I agree that this should not block Dillo-2.1 . Just think of it as the Moby Dick to my Captain Ahab. From Hell's heart I stab at thee, freed struct DilloImage! Dillo-2.1 *will* rock, whether or not this is fixed.

May 23, 2009

[rutkowski.adrian1@gmail.com: Re: [Dillo-dev] Dillo failed to compile with GCC 4.4.0]

by jcid＠dillo.org

On Fri, May 22, 2009 at 06:51:16PM +0000, corvid wrote: > No need. Adrian went through a few rounds of sending me the error > msgs, I committed some consts, and then he reported: > > > Success! :) > > Now it compiles and runs whitout glitches. Excellent! -- Cheers Jorge.-

May 23, 2009

Latest on invalid image callback references

by jcid＠dillo.org

On Tue, May 19, 2009 at 07:34:30AM +0100, Jeremy Henty wrote: > > The valgrind logs are still showing invalid memory accesses that > happen when image callbacks refer to DilloImages that have been freed > by a_Web_free(). This is after the imgref patch was applied, since > when I've been running an essentially unpatched Dillo. My image > tracing still shows that the image pipeline sometimes continues to run > after a_Web_free() frees the image it is attached to. As far as I can > tell there has been no recent change in the behaviour of this problem. I see. > I have just applied a patch I was considering earlier that makes > a_Cache_stop_client() close the client. I've attached it for > comments, but I won't suggest applying it until we can see whether > it's made any difference. That will take a few days, as I still > haven't found a reliable way to reproduce the bug so I will just have > to wait a few days and see if it stops appearing in the logs. Please > comment on whether this patch is the right way to fix this. A long time ago when coding the bindings for the new imgbuf, I seem to remember seeing a corner case that would leak. Back then the priority was to get the port into a working state so it was left for future fixing (as many other things). Since then, the image code has undergone major changes and cleanups and it may take some effort to get to this bug. One chance I checked yesterday was a mismatched image type (e.g. PNG served as GIF) but it wasn't the cause. As we're almost done with rc1 for the dillo-2.1 release. I'd prefer to work on it after the release, or to dedicate more time to it only if there's a suitable window in the release process. A huge step forward would be to find a way to repoduce the bug. WRT the patch, it may be the fix, but there's a need to check for the current semantic of stop_clients in all cases. This function may only remove the client but not shut the decoding process (think of a repush for instance), and BTW some parts of the code use a flag (as "force"). My guess is that this semantic is still fuzzy so it may be necessary to study the whole, define an accurate behaviour, code accordingly, check the corner cases and to test in the "wild". -- Cheers Jorge.-

May 22, 2009

Dillo failed to compile with GCC 4.4.0

by jcid＠dillo.org

On Fri, May 22, 2009 at 06:53:09PM +0200, Adrian Rutkowski wrote: > Hi, I'm trying to compile latest dillo hg snapshot with GCC 4.4.0 > and got this error: > > g++ -DHAVE_CONFIG_H -I. -I.. -I/usr/local/include -g -O2 -Wall -W > -Wno-unused-parameter -fno-rtti -fno-exceptions -MT tablecell.o -MD -MP > -MF .deps/tablecell.Tpo -c -o tablecell.o tablecell.cc > tablecell.cc: In member function ???virtual void > dw::TableCell::wordWrap(int)???: > tablecell.cc:55: error: invalid conversion from ???const char*??? to ???char*??? > make[2]: *** [tablecell.o] B????d 1 > make[2]: Opuszczenie katalogu `/home/adrian/src/dillo/dw' > make[1]: *** [all-recursive] B????d 1 > make[1]: Opuszczenie katalogu `/home/adrian/src/dillo' > make: *** [all] B????d 2 > > System is Arch Linux with latest gcc package (gcc-4.4.0-2). Can you provide temporary access (account/passwd) to a machine with gcc-4.4.0-2? I'd gladly take a look into it (you can send me the account/passwd encrypted with dillo.org/authors/jcid.gpg) Unfortunately the proposed patch has severe flaws with memory handling. -- Cheers Jorge.-

May 22, 2009

Dillo failed to compile with GCC 4.4.0

by rutkowski.adrian1＠gmail.com

On Fri, May 22, 2009 at 05:34:20PM +0000, corvid wrote: > Does this compile cleanly for you: > > diff -r 717e6ced084f src/uicmd.cc > --- a/src/uicmd.cc Fri May 22 17:16:18 2009 +0000 > +++ b/src/uicmd.cc Fri May 22 17:32:05 2009 +0000 > @@ -694,8 +694,8 @@ static char *UIcmd_make_save_filename(co > static char *UIcmd_make_save_filename(const char *pathstr) > { > size_t MaxLen = 64; > - char *FileName, *name, *newname, *o, *n; > - const char *dir = a_UIcmd_get_save_dir(); > + char *FileName, *newname, *o, *n; > + const char *name, *dir = a_UIcmd_get_save_dir(); > > if ((name = strrchr(pathstr, '/'))) { > if (strlen(++name) > MaxLen) { Yes that fixed it, now it stops here: g++ -DHAVE_CONFIG_H -I. -I.. -DDILLORC_SYS='"/usr/local/etc/dillorc"' -I/usr/local/include -I.. -I/usr/include/libpng12 -I/usr/local/include -Wno-non-virtual-dtor -g -O2 -Wall -W -Wno-unused-parameter -fno-rtti -fno-exceptions -MT html.o -MD -MP -MF .deps/html.Tpo -c -o html.o html.cc html.cc: In function ?void Html_tag_open_meta(DilloHtml*, const char*, int)?: html.cc:2812: error: invalid conversion from ?const char*? to ?char*? make[1]: *** [html.o] B??d 1 make[1]: Opuszczenie katalogu `/home/adrian/src/dillo/src' make: *** [all-recursive] B??d 1

May 22, 2009

Dillo failed to compile with GCC 4.4.0

by rutkowski.adrian1＠gmail.com

g++ -DHAVE_CONFIG_H -I. -I.. -DDILLORC_SYS='"/usr/local/etc/dillorc"' -I/usr/local/include -I.. -I/usr/include/libpng12 -I/usr/local/include -Wno-non-virtual-dtor -g -O2 -Wall -W -Wno-unused-parameter -fno-rtti -fno-exceptions -MT uicmd.o -MD -MP -MF .deps/uicmd.Tpo -c -o uicmd.o uicmd.cc uicmd.cc: In function ?char* UIcmd_make_save_filename(const char*)?: uicmd.cc:700: error: invalid conversion from ?const char*? to ?char*? make[3]: *** [uicmd.o] B??d 1 make[3]: Opuszczenie katalogu `/home/adrian/src/dillo/src' make[2]: *** [all-recursive] B??d 1 make[2]: Opuszczenie katalogu `/home/adrian/src/dillo/src' make[1]: *** [all-recursive] B??d 1 make[1]: Opuszczenie katalogu `/home/adrian/src/dillo' make: *** [all] B??d 2

May 22, 2009

Dillo failed to compile with GCC 4.4.0

by corvid＠lavabit.com

Tim wrote: > > Hi, I'm trying to compile latest dillo hg snapshot with GCC 4.4.0 > > and got this error: > > [...] > > There is already a patch. Unfortunately it has not been integrated > into mainline yet. See thread ``[patch] GCC v4.4''. I'll commit the tablecell line in a moment. Then someone can tell me exactly what error msg they get with form.cc and so on...

May 22, 2009

Dillo failed to compile with GCC 4.4.0

by tim.nieradzik＠gmx.de

> Hi, I'm trying to compile latest dillo hg snapshot with GCC 4.4.0 > and got this error: > [...] There is already a patch. Unfortunately it has not been integrated into mainline yet. See thread ``[patch] GCC v4.4''. --Tim

May 22, 2009