- Dillo-dev - lists.mailman3.com

Patch for xembed (updated for resize)

by podarcis＠lavabit.com

Hi, well the aim would be to contribute to dillo to make it embeddable by X11 applications. If you want to achieve this with "standard" XEMBED protocol only, I am off -that's not my domain... I have tested/debugged the current code with 'tabbed' (as you suggested) and found out, that it works even better if the ClientMessage code (and this is the only relevant code here) is removed. tabbed also showed some glitches with the current code: - dillo's File menu opens at different positions after moving - clicking in the grey space right to dillo's 'Tools' icon seems to crash something... So I removed the whole thing. Did you try my patch? It certainly works not only from FLTK applications. You can verify that, if you try to embed dillo into any window on your desktop e.g. a gnome terminal (use xwininfo to find out its xid). I do not claim that my patch is complete already, but it may at least be used as a starting point. At least it has the advantage, that the embedder does not need to do anything, except passing an xid. If keeping the old code is your only concern, then I can re-integrate it into my patch again - it has no effect on it at all. Cheers chris > Hi, > > On Sat, Dec 15, 2012 at 12:04:35PM +0100, podarcis wrote: >> Hi, >> >> here is an updated version making resize work. >> It still works with tabbed too. >> Note that I have had no knowledge of X or xembed at all, so all >> changes come from experimenting... >> I removed large parts of the existing code, because they did nothing >> in my environment. >> >> Please have a look at it. > > hm, the patch basically removes support for the xembed protocol. Is > it meant as a diagnostic patch to investigate a problem, or meant > for inclusion into the dillo repository? > It won't work for the latter case. You can't just remove code that > is not needed for your specific case. > If you just want to get embedding of dillo into an fltk based window > working, you would need to send the proper xembed X11 event from your > embedding fltk program. Look at what tabbed is doing. > I can also dig into my old stuff, I think I once had a minimal fltk > based embedder working. > > Cheers, > Johannes > > _______________________________________________ > Dillo-dev mailing list > Dillo-dev at dillo.org > http://lists.auriga.wearlab.de/cgi-bin/mailman/listinfo/dillo-dev >

Dec. 20, 2012

Extensions for hyphenation

by corvid＠lavabit.com

The text spills over into the following cell in <table border=1> <tr> <td>W d<whatever>aaaaaaaaaaaa*<br> <td>WW d<whatever>aaaaaaaaaaaa*<br> <td>WWW d<whatever>aaaaaaaaaaaa*<br> <td>WWWW d<whatever>aaaaaaaaaaaa*<br> <td>WWWWW d<whatever>aaaaaaaaaaaa*<br> <td>WWWWWW d<whatever>aaaaaaaaaaaa*<br> </table> if I make the browser window narrowish.

Dec. 20, 2012

config.h.in and configure.in

by jcid＠dillo.org

On Wed, Dec 19, 2012 at 01:24:21PM +0000, Jeremy Henty wrote: > > I thought that we agreed[1] to remove config.h.in from the repository > and to rename configure.in to configure.ac . Can I make these changes > or is more discussion needed? Go ahead! -- Cheers Jorge.-

Dec. 19, 2012

config.h.in and configure.in

by onepoint＠starurchin.org

I thought that we agreed[1] to remove config.h.in from the repository and to rename configure.in to configure.ac . Can I make these changes or is more discussion needed? Regards, Jeremy Henty [1]: http://lists.auriga.wearlab.de/pipermail/dillo-dev/2012-December/009618.html

Dec. 19, 2012

Patch for xembed (updated for resize)

by johannes.hofmann＠gmx.de

Hi, On Sat, Dec 15, 2012 at 12:04:35PM +0100, podarcis wrote: > Hi, > > here is an updated version making resize work. > It still works with tabbed too. > Note that I have had no knowledge of X or xembed at all, so all > changes come from experimenting... > I removed large parts of the existing code, because they did nothing > in my environment. > > Please have a look at it. hm, the patch basically removes support for the xembed protocol. Is it meant as a diagnostic patch to investigate a problem, or meant for inclusion into the dillo repository? It won't work for the latter case. You can't just remove code that is not needed for your specific case. If you just want to get embedding of dillo into an fltk based window working, you would need to send the proper xembed X11 event from your embedding fltk program. Look at what tabbed is doing. I can also dig into my old stuff, I think I once had a minimal fltk based embedder working. Cheers, Johannes

Dec. 19, 2012

dpid patches

by p37sitdu＠lavabit.com

First patch is for removing expired TODO item, just resending it. Second patch is new, for dpid.c. Function fill_services_list has a bug: on line 463 there is a check *services_list != NULL. If it is true, then -1 is returned, but dpidrc_stream is not closed. Error message on line 456 says "popen failed", should be "fopen failed". fill_services_list checks for availability of user_dpidir and sys_dpidir, but after line 449 these variables are not used. They are just dFreed in the end. Looks like code was just copypasted from register_all. fill_services_list is only called from main.c:248 and dpid.c:807 of dpid.c. register_all is called from main.c:235 and dpid.c:806. So fill_services_list is always called after register_all and doing the same check is useless. On line 503 there is a conversion of services_alpha_comp to type (dCompareFunc). I changed the function services_alpha_comp to the usual style of writing comparison functions so no conversion is required. In the current state it is undefined behaviour. [1] [1] http://stackoverflow.com/questions/559581/casting-a-function-pointer-to-ano…

Dec. 18, 2012

[patch] Gopher DPI

by p37sitdu＠lavabit.com

Fixed memory leak related to dGetline.

Dec. 17, 2012

[patch] Gopher DPI

by p37sitdu＠lavabit.com

Small changes - Added 'p' filetype for images (used by some gopher servers) - Use dGetline instead of getline

Dec. 17, 2012

Tooltip bugs revisited

by jcid＠dillo.org

On Tue, Dec 11, 2012 at 04:30:59AM +0000, corvid wrote: > Jorge wrote: > > So far (and despite the warning inside FLTK), I've experimented > > with a custom tooltip handling class, and a couple custom UI > > widgets, that seem to work well together (i.e. solve the problem > > without the workaround :) ). > > I see that this code is derived from the code at > http://seriss.com/people/erco/fltk/#SliderTooltip > I think we should mention that at the tops of the tipwin files and > in AUTHORS. Yes, I wrote to Greg and got his answer stating that all the code there is public domain, and that he'll soon add some lines declaring it so in his "Cheat Page". -- Cheers Jorge.-

Dec. 17, 2012

Dpi_connect_socket retry

by jcid＠dillo.org

Hi, Finally I had the time to review this issue in more detail... On Sun, Dec 09, 2012 at 02:58:43PM +0400, p37sitdu at lavabit.com wrote: > Dpi_connect_socket function from src/IO/dpi.c (also duplicated in > test/cookies.c) has retry argument. It is set to TRUE on first call, > and Dpi_connect_socket calls itself recursively with retry == FALSE in > case of error. > > This retry function is not working. If connection fails, at line 645 > sock_fd is set, but ret is still set to -1. Even if the second attempt > to connect is successful, -1 is returned and conected sock_fd is lost. > > It is possible to fix it by adding "ret = sock_fd", but I think it is > better to just remove retries completely. It is unlikely that > connecting using loopback interface will fail on first try and succeed > on retry. Yes, this code is obsolete now. It was meant for Unix domain sockets (when dpis/dpid used them). Sometimes the filename lingered in the filesystem after a dpi or server crash. With unlink&retry it was possible to recover then. Details in patchset: 7faa2c7a544f (Nov, 2009 ;) Removed in the repo (see below). > Also, comment says "the server closes sock_fd on error". However, > sock_fd should still be closed by client to free allocated file > descriptor. OS don't know if Dillo is going to read from it (in which > case error such as ECONNRESET should happen) so file descriptor can't > be reused. > > File descriptor should be closed in case of error in a_Dpip_build_cmd > too. The file descriptor is properly closed by its CCC afterwards (Visible by defining VERBOSE in chain.c). The CCC manages the error propagation/handling, in both the requesting and answer branches. > Patch to fix this function and its calls in src/IO/dpi.c and > test/cookies.c is attached. No need to fix, it works properly. -- Cheers Jorge.-

Dec. 16, 2012