September 2008 - Dillo-dev - lists.mailman3.com

segfault -- callback + closed window?

by corvid＠lavabit.com

Oh, and I want to make it clear that this has nothing to do with the patch Johannes just made which I haven't applied yet.

Sept. 29, 2008

segfault -- callback + closed window?

by corvid＠lavabit.com

I hit ',' a few times, going back past a page with a big progressive jpeg webcomic, and hit ^Q to close the window while dillo was thinking its way past that image. Got: Program received signal SIGSEGV, Segmentation fault. [Switching to Thread 16384 (LWP 2782)] 0x080fc708 in fltk::Widget::redraw (this=0x81cb680, flags=8 '\b') at Widget.cxx:575 575 widget->damage_ |= DAMAGE_CHILD; (gdb) bt #0 0x080fc708 in fltk::Widget::redraw (this=0x81cb680, flags=8 '\b') at Widget.cxx:575 #1 0x0804f679 in UI::button_set_sens (this=0x82ce9d8, btn=UI_STOP, sens=0) at ui.cc:1109 #2 0x08054c59 in a_UIcmd_set_buttons_sens_cb (vbw=0x81ad858) at uicmd.cc:835 #3 0x080e47f9 in fltk::wait () at TabGroup.cxx:116 #4 0x080e489d in fltk::run () at TabGroup.cxx:116 #5 0x0804efab in main (argc=1, argv=0xbffffb44) at dillo.cc:127 Maybe that's just a part of life with callbacks...

Sept. 29, 2008

patch: changeLinkColor

by Johannes.Hofmann＠gmx.de

On Sun, Sep 28, 2008 at 08:28:49PM +0000, corvid wrote: > Johannes wrote: > > BTW. with both implementations I now see issues with this example: > > > > <html> > > <body> > > <a href="http://www.dillo.org">dillo</a> > > <a href="http://slashdot.org">slashdot</a> > > <a href="http://www.dillo.org">dillo</a> > > </body> > > </html> > > > > When opening the first link in a new window, it doesn't change color > > in the original one. > > When opening it in a tab, the first one changes color. After a > > reload also the second one changes color. > > Am I missing something? > > Middle click changes the color for me whether it opens a tab or a window. > Sorry, only now I noticed that it only happens when opening the new window/tab via the "Link Options" menu. Cheers, Johannes

Sept. 29, 2008

patch: changeLinkColor

by Johannes.Hofmann＠gmx.de

On Sun, Sep 28, 2008 at 04:40:53PM -0400, Jorge Arellano Cid wrote: > On Sun, Sep 28, 2008 at 09:20:17PM +0200, Johannes Hofmann wrote: > > On Sun, Sep 28, 2008 at 02:54:56PM -0400, Jorge Arellano Cid wrote: > > > On Sun, Sep 28, 2008 at 05:55:10PM +0000, corvid wrote: > > > > Encouraged by the idea that I am going in the right direction, I have > > > > made a new one with spaceStyle: > > > > > > Funny how we were working on the same patch a the same time! > > > > > > I had an ancient version, and gave it a review. > > > The link search is linear inside textblock. A linux gazette page > > > showed more than 33000 words to color a link. On bigger pages it may be worst. > > > It depends on the structure. For instance our bookmarks uses tables > > > and the link is usually found in less than 10 words. > > > > > > The patch in CVS reduce the 35000 steps to 1500 or so (the > > > sorted link list is sparse, so there's a penalty for looking > > > for an actual link to compare to). > > > > > > > Not sure I understand this optimization. > > It is assuming that link numbers are always increasing, right? > > Right. > > > Can we assume this in dw/*? > > I checked html.cc and x_link is crescent. Not sure about the rest. > > > src/html.cc seems to be implemented currently in way that this is > > true. But perhaps we could/should reuse link numbers for equal > > destination urls in the future? > > In that case it'd need a revision. > > > Or someone might just choose hashes of the url string as link > > number. > > > > BTW. with both implementations I now see issues with this example: > > > > <html> > > <body> > > <a href="http://www.dillo.org">dillo</a> > > <a href="http://slashdot.org">slashdot</a> > > <a href="http://www.dillo.org">dillo</a> > > </body> > > </html> > > > > When opening the first link in a new window, it doesn't change color > > in the original one. > > When opening it in a tab, the first one changes color. After a > > reload also the second one changes color. > > Am I missing something? > > Yes, the link color change is meant as a visual cue for the user > to know the click is being processed. We could leave the link alone > and change the mouse cursor instead. > > The link not changing color when openning it with the menu > is IMO a bug, I'm looking for... > > BTW, the linear search patch is much simpler and as link > clicking is not a performance critical task it could be left > implemented in O(n). I prefer linear search. Once it really becomes a performance problem we can look for an optimization. Cheers, Johannes

Sept. 29, 2008

Window close dialog

by Johannes.Hofmann＠gmx.de

On Mon, Sep 29, 2008 at 11:12:39AM -0400, Jorge Arellano Cid wrote: > On Mon, Sep 29, 2008 at 04:03:55PM +0200, Johannes Hofmann wrote: > > Hi, > > > > On Mon, Sep 29, 2008 at 09:48:58AM -0400, Jorge Arellano Cid wrote: > > > Hi, > > > > > > Can somebody please make the dialog for closing a window > > > with tabs open? > > > > "Close Window" only closes the current tab anyway. > > Yes. > > > "Exit Dillo" closes all windows including tabs. > > So, do you think about a dialog in the "Exit Dillo" case when > > there is more than one tab or more than one window or both? > > Both, plus the window manager's "close window" if there're open Tabs. > > > Do we have the number of open windows somewhere? > > In bw.cc's num_bws. > Ok, please check attached patch. It relies on the current browser window to be stored in a window's user_data. We may want to reconsider this, but certainly not before the release. Cheers, Johannes

Sept. 29, 2008

what crashes remain?

by corvid＠lavabit.com

Jorge wrote: > On Mon, Sep 29, 2008 at 02:09:36PM +0000, corvid wrote: > > Jorge wrote: > > > On Sun, Sep 28, 2008 at 12:52:47AM +0000, corvid wrote: > > > > Playing with the file menu, I see that Alt-W, Alt-E, Alt-X, Alt-C > > > > all segfault. I thought somebody said something (that I didn't pay > > > > attention to) about the file menu since tabs were added, but if so, > > > > I can't find that mail. > > > > > > Please check the patch in CVS. > > > > > > FLTK2 has a bug in that it regards the first menu item letter as > > > a shortcut. > > > > > > Note: I tried with invisible shortcuts doing nothing, > > > removing shortcuts and all that stuff, until I found this way > > > to workaround the problem. > > > > > > Note2: Alt-W and Alt-X are accelerators, so they're OK. > > > > > > AFAIS there's no more segfault here. > > > > Alt-W and Alt-X still segfault for me. > > Hmm... > > What fltk version are you using? > > Anyway, as a workaround, try removing the ampersand in the > labels for "Close Window" and "Exit Dillo", in UI::make_menubar(). The latest snapshot, 6305. Removing the ampersands == no more segfaults for me.

Sept. 29, 2008

Window close dialog

by jcid＠dillo.org

On Mon, Sep 29, 2008 at 04:03:55PM +0200, Johannes Hofmann wrote: > Hi, > > On Mon, Sep 29, 2008 at 09:48:58AM -0400, Jorge Arellano Cid wrote: > > Hi, > > > > Can somebody please make the dialog for closing a window > > with tabs open? > > "Close Window" only closes the current tab anyway. Yes. > "Exit Dillo" closes all windows including tabs. > So, do you think about a dialog in the "Exit Dillo" case when > there is more than one tab or more than one window or both? Both, plus the window manager's "close window" if there're open Tabs. > Do we have the number of open windows somewhere? In bw.cc's num_bws. -- Cheers Jorge.-

Sept. 29, 2008

what crashes remain?

by jcid＠dillo.org

On Mon, Sep 29, 2008 at 02:09:36PM +0000, corvid wrote: > Jorge wrote: > > On Sun, Sep 28, 2008 at 12:52:47AM +0000, corvid wrote: > > > Playing with the file menu, I see that Alt-W, Alt-E, Alt-X, Alt-C > > > all segfault. I thought somebody said something (that I didn't pay > > > attention to) about the file menu since tabs were added, but if so, > > > I can't find that mail. > > > > Please check the patch in CVS. > > > > FLTK2 has a bug in that it regards the first menu item letter as > > a shortcut. > > > > Note: I tried with invisible shortcuts doing nothing, > > removing shortcuts and all that stuff, until I found this way > > to workaround the problem. > > > > Note2: Alt-W and Alt-X are accelerators, so they're OK. > > > > AFAIS there's no more segfault here. > > Alt-W and Alt-X still segfault for me. Hmm... What fltk version are you using? Anyway, as a workaround, try removing the ampersand in the labels for "Close Window" and "Exit Dillo", in UI::make_menubar(). > What's an accelerator? Kind of a shortcut that you set with an ampersand in the label of an fltk::Item. Disclaimer: "accelerator" may not be the correct name for it. -- Cheers Jorge.-

Sept. 29, 2008

what crashes remain?

by corvid＠lavabit.com

Jorge wrote: > On Sun, Sep 28, 2008 at 12:52:47AM +0000, corvid wrote: > > Playing with the file menu, I see that Alt-W, Alt-E, Alt-X, Alt-C > > all segfault. I thought somebody said something (that I didn't pay > > attention to) about the file menu since tabs were added, but if so, > > I can't find that mail. > > Please check the patch in CVS. > > FLTK2 has a bug in that it regards the first menu item letter as > a shortcut. > > Note: I tried with invisible shortcuts doing nothing, > removing shortcuts and all that stuff, until I found this way > to workaround the problem. > > Note2: Alt-W and Alt-X are accelerators, so they're OK. > > AFAIS there's no more segfault here. Alt-W and Alt-X still segfault for me. What's an accelerator?

Sept. 29, 2008

Window close dialog

by Johannes.Hofmann＠gmx.de

Hi, On Mon, Sep 29, 2008 at 09:48:58AM -0400, Jorge Arellano Cid wrote: > Hi, > > Can somebody please make the dialog for closing a window > with tabs open? "Close Window" only closes the current tab anyway. "Exit Dillo" closes all windows including tabs. So, do you think about a dialog in the "Exit Dillo" case when there is more than one tab or more than one window or both? Do we have the number of open windows somewhere? Cheers, Johannes

Sept. 29, 2008