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

EAGAIN loop in IO_read

by furaisanjin＠gmail.com

This misleads, so I clarify. 2009/11/12 furaisanjin <furaisanjin(a)gmail.com>: > I have no idea but the problem did happen in one hour but now it's > very difficult to make it happen. I haven't seen it since I applied the patch. Regards, furaisanjin

Nov. 12, 2009

EAGAIN loop in IO_read

by furaisanjin＠gmail.com

2009/11/12 Jorge Arellano Cid <jcid(a)dillo.org>: > On Thu, Nov 12, 2009 at 07:54:25AM +0900, furaisanjin wrote: >> Hi. >> >> My problem also seems to be fixed because it's very harder to >> reproduce the problem now. > > ?Is still possible to trigger the problem (but very hard)? I have no idea but the problem did happen in one hour but now it's very difficult to make it happen. >> By the way, I'm a bit wondering this code >> and I suppose something like this. >> >> --- dillo/src/IO/dpi.c ?2009-11-11 05:51:28.000000000 +0900 >> +++ dillo-test/src/IO/dpi.c ? ? 2009-11-11 22:05:42.000000000 +0900 >> @@ -793,12 +793,15 @@ char *a_Dpi_send_blocking_cmd(const char >> >> ? ? if ((sock_fd = Dpi_connect_socket(server_name, TRUE)) == -1) { >> ? ? ? ?MSG_ERR("[a_Dpi_send_blocking_cmd] Can't connect to server.\n"); >> - ? } else if (Dpi_blocking_write(sock_fd, cmd, strlen(cmd)) == -1) { >> - ? ? ?MSG_ERR("[a_Dpi_send_blocking_cmd] Can't send message.\n"); >> - ? } if ((ret = Dpi_blocking_read(sock_fd)) == NULL) { >> - ? ? ?MSG_ERR("[a_Dpi_send_blocking_cmd] Can't read message.\n"); >> + ? } else { >> + ? ? if (Dpi_blocking_write(sock_fd, cmd, strlen(cmd)) == -1) { >> + ? ? ? MSG_ERR("[a_Dpi_send_blocking_cmd] Can't send message.\n"); >> + ? ? } >> + ? ? if ((ret = Dpi_blocking_read(sock_fd)) == NULL) { >> + ? ? ? MSG_ERR("[a_Dpi_send_blocking_cmd] Can't read message.\n"); >> + ? ? } >> + ? ? Dpi_close_fd(sock_fd); >> ? ? } >> - ? Dpi_close_fd(sock_fd); > > ?Dpi_close_fd() checks fd != -1 so AFAIS it's OK. I meant Dpi_blocking_read as well. Regards, furaisanjin

Nov. 12, 2009

modify bookmarks (Re: [Dillo-dev] EAGAIN loop in IO_read)

by corvid＠lavabit.com

Jorge wrote: > On Thu, Nov 12, 2009 at 12:08:33AM +0000, corvid wrote: > > Jorge wrote: > > > BTW, I chasing a very weird bug here. I can't modify a bookmark > > > unless I comment closing a stream on an unrelated file! > > > > > > This is: > > > > > > @@ -417,8 +417,10 @@ static int Dpi_read_comm_keys(int *port) > > > SharedKey[i] = 0; > > > ret = 1; > > > } > > > - if (In) > > > - fclose(In); > > > > > > > > > Then it works! :-P > > > > > > @all: The question is, is this happening only in my system or do you > > > see the problem as well. In other words: can you modify a bookmark? > > > > I don't normally use bookmarks, but if I try it, I click submit, get > > Nav_open_url: new url='dpi:/bm/modify?operation=modify&submit=submit.&s0=on&url1=on' > > Nav_open_url: new url='dpi:/bm/modify' > > > > and...nothing else seems to happen. > > Exactly as here... > > and if you apply the above patch? Yup, I get a new page for updating the title.

Nov. 12, 2009

modify bookmarks (Re: [Dillo-dev] EAGAIN loop in IO_read)

by jcid＠dillo.org

On Thu, Nov 12, 2009 at 12:08:33AM +0000, corvid wrote: > Jorge wrote: > > BTW, I chasing a very weird bug here. I can't modify a bookmark > > unless I comment closing a stream on an unrelated file! > > > > This is: > > > > @@ -417,8 +417,10 @@ static int Dpi_read_comm_keys(int *port) > > SharedKey[i] = 0; > > ret = 1; > > } > > - if (In) > > - fclose(In); > > > > > > Then it works! :-P > > > > @all: The question is, is this happening only in my system or do you > > see the problem as well. In other words: can you modify a bookmark? > > I don't normally use bookmarks, but if I try it, I click submit, get > Nav_open_url: new url='dpi:/bm/modify?operation=modify&submit=submit.&s0=on&url1=on' > Nav_open_url: new url='dpi:/bm/modify' > > and...nothing else seems to happen. Exactly as here... and if you apply the above patch? -- Cheers Jorge.-

Nov. 12, 2009

modify bookmarks (Re: [Dillo-dev] EAGAIN loop in IO_read)

by corvid＠lavabit.com

Jorge wrote: > BTW, I chasing a very weird bug here. I can't modify a bookmark > unless I comment closing a stream on an unrelated file! > > This is: > > @@ -417,8 +417,10 @@ static int Dpi_read_comm_keys(int *port) > SharedKey[i] = 0; > ret = 1; > } > - if (In) > - fclose(In); > > > Then it works! :-P > > @all: The question is, is this happening only in my system or do you > see the problem as well. In other words: can you modify a bookmark? I don't normally use bookmarks, but if I try it, I click submit, get Nav_open_url: new url='dpi:/bm/modify?operation=modify&submit=submit.&s0=on&url1=on' Nav_open_url: new url='dpi:/bm/modify' and...nothing else seems to happen.

Nov. 12, 2009

EAGAIN loop in IO_read

by jcid＠dillo.org

On Thu, Nov 12, 2009 at 07:54:25AM +0900, furaisanjin wrote: > Hi. > > My problem also seems to be fixed because it's very harder to > reproduce the problem now. Is still possible to trigger the problem (but very hard)? > By the way, I'm a bit wondering this code > and I suppose something like this. > > --- dillo/src/IO/dpi.c 2009-11-11 05:51:28.000000000 +0900 > +++ dillo-test/src/IO/dpi.c 2009-11-11 22:05:42.000000000 +0900 > @@ -793,12 +793,15 @@ char *a_Dpi_send_blocking_cmd(const char > > if ((sock_fd = Dpi_connect_socket(server_name, TRUE)) == -1) { > MSG_ERR("[a_Dpi_send_blocking_cmd] Can't connect to server.\n"); > - } else if (Dpi_blocking_write(sock_fd, cmd, strlen(cmd)) == -1) { > - MSG_ERR("[a_Dpi_send_blocking_cmd] Can't send message.\n"); > - } if ((ret = Dpi_blocking_read(sock_fd)) == NULL) { > - MSG_ERR("[a_Dpi_send_blocking_cmd] Can't read message.\n"); > + } else { > + if (Dpi_blocking_write(sock_fd, cmd, strlen(cmd)) == -1) { > + MSG_ERR("[a_Dpi_send_blocking_cmd] Can't send message.\n"); > + } > + if ((ret = Dpi_blocking_read(sock_fd)) == NULL) { > + MSG_ERR("[a_Dpi_send_blocking_cmd] Can't read message.\n"); > + } > + Dpi_close_fd(sock_fd); > } > - Dpi_close_fd(sock_fd); Dpi_close_fd() checks fd != -1 so AFAIS it's OK. BTW, I chasing a very weird bug here. I can't modify a bookmark unless I comment closing a stream on an unrelated file! This is: @@ -417,8 +417,10 @@ static int Dpi_read_comm_keys(int *port) SharedKey[i] = 0; ret = 1; } - if (In) - fclose(In); Then it works! :-P @all: The question is, is this happening only in my system or do you see the problem as well. In other words: can you modify a bookmark? -- Cheers Jorge.-

Nov. 12, 2009

EAGAIN loop in IO_read

by furaisanjin＠gmail.com

Hi. My problem also seems to be fixed because it's very harder to reproduce the problem now. By the way, I'm a bit wondering this code and I suppose something like this. --- dillo/src/IO/dpi.c 2009-11-11 05:51:28.000000000 +0900 +++ dillo-test/src/IO/dpi.c 2009-11-11 22:05:42.000000000 +0900 @@ -793,12 +793,15 @@ char *a_Dpi_send_blocking_cmd(const char if ((sock_fd = Dpi_connect_socket(server_name, TRUE)) == -1) { MSG_ERR("[a_Dpi_send_blocking_cmd] Can't connect to server.\n"); - } else if (Dpi_blocking_write(sock_fd, cmd, strlen(cmd)) == -1) { - MSG_ERR("[a_Dpi_send_blocking_cmd] Can't send message.\n"); - } if ((ret = Dpi_blocking_read(sock_fd)) == NULL) { - MSG_ERR("[a_Dpi_send_blocking_cmd] Can't read message.\n"); + } else { + if (Dpi_blocking_write(sock_fd, cmd, strlen(cmd)) == -1) { + MSG_ERR("[a_Dpi_send_blocking_cmd] Can't send message.\n"); + } + if ((ret = Dpi_blocking_read(sock_fd)) == NULL) { + MSG_ERR("[a_Dpi_send_blocking_cmd] Can't read message.\n"); + } + Dpi_close_fd(sock_fd); } - Dpi_close_fd(sock_fd); return ret; } Regards, furaisanjin

Nov. 11, 2009

weird font

by jcid＠dillo.org

On Tue, Nov 10, 2009 at 11:06:30PM +0100, Johannes Hofmann wrote: > Hi Jorge, > > On Tue, Nov 10, 2009 at 06:46:45PM -0300, Jorge Arellano Cid wrote: > > Hi, > > > > On some sites I get weird fonts. Today I got to isolate the > > problem. See attached file. Here it displays like this: > > > > http://www.dillo.org/test/xmp2.png > > > > The embedded CSS says: > > > > pre { font-family: MS Courier New, Courier, monospace; } > > > > If I remove "MS Courier New", it works OK. > > Any clues? > > There are multiple issues with font selection: > > * The CSS parser only returns the part until the first blank > ('MS' in this case). > * StyleEngine currently does not handle comma separated font name > lists (see \todo comment). > * ::fltk::font(name, fa); in fltkplatform.cc does not offer a way to > determine whether a font with that name actually exists on the > system. Instead if no exact match can be found, it guesses a > replacement font. This makes it impossible for dillo to choose the > right one from a given list. > > In your specific case, fltk gets the string 'MS' and on your system > it seems to find some funny font that has a similar name. Thanks for the detailed explanation. -- Cheers Jorge.-

Nov. 11, 2009

weird font

by marc＠marcengelhardt.com

Jorge wrote: If I remove "MS Courier New", it works OK. Any clues? I think the problem is the missing quote signs ("") before and after the font name (MS Courier New). I can be wrong, but I think that Dillo doesn't recognizes that the three words (MS, Courier, New), which normally builds the font, must be combined to build a valid font. The problem can be fixed if you add a feature to Dillo, which let Dillo recognizes that a string in a CSS rule (before a comma) represents a valid font and should be applied to the elements on the page, which are affected by the CSS rule. So, Dillo should be recognizes fonts, which are one, two or three words in length. Please keep in mind, that font names, which are between the quote signs, must also be recognized by Dillo.

Nov. 10, 2009

weird font

by Johannes.Hofmann＠gmx.de

Hi Jorge, On Tue, Nov 10, 2009 at 06:46:45PM -0300, Jorge Arellano Cid wrote: > Hi, > > On some sites I get weird fonts. Today I got to isolate the > problem. See attached file. Here it displays like this: > > http://www.dillo.org/test/xmp2.png > > The embedded CSS says: > > pre { font-family: MS Courier New, Courier, monospace; } > > If I remove "MS Courier New", it works OK. > Any clues? There are multiple issues with font selection: * The CSS parser only returns the part until the first blank ('MS' in this case). * StyleEngine currently does not handle comma separated font name lists (see \todo comment). * ::fltk::font(name, fa); in fltkplatform.cc does not offer a way to determine whether a font with that name actually exists on the system. Instead if no exact match can be found, it guesses a replacement font. This makes it impossible for dillo to choose the right one from a given list. In your specific case, fltk gets the string 'MS' and on your system it seems to find some funny font that has a similar name. Cheers, Johannes

Nov. 10, 2009