On Wed, Apr 27, 2011 at 08:35:36PM +0000, corvid wrote:
I wrote:
Rob wrote:
BTW, building the latest 1.3 version with -O0 makes the segfault disappear.
Ah! I can reproduce this now with -O2. I'll see what's going on...
I can also reproduce it with -O2.
Does the attached fix it for you?
diff -r 165b50e3f996 src/menu.cc --- a/src/menu.cc Wed Apr 27 19:56:21 2011 +0000 +++ b/src/menu.cc Wed Apr 27 20:34:25 2011 +0000 @@ -671,6 +671,7 @@ FL_MENU_RADIO|FL_MENU_DIVIDER,0,0,0,0}, {"small icons", 0,Menu_panel_change_cb,(void*)10, FL_MENU_TOGGLE,0,0,0,0}, + {0,0,0,0,0,0,0,0,0}, {0,0,0,0,0,0,0,0,0} };
Yes it does! And I also checked that regressing the repo to the version before the submenu patch (e816e06ee901) also "fixes" it. @Rob: would you mind double check the patch is correctly applied on src/menu.cc:674?, then recompile, and also make sure you run the newly compiled dillo and not the system wide one? -- Cheers Jorge.-