https://bugzilla.novell.com/show_bug.cgi?id=732898 https://bugzilla.novell.com/show_bug.cgi?id=732898#c3 --- Comment #3 from Wolfgang Rosenauer <wolfgang@rosenauer.org> 2011-11-28 09:46:58 UTC --- (From update of attachment 464260)
diff -ur mozilla.orig/accessible/src/atk/nsApplicationAccessibleWrap.cpp mozilla/accessible/src/atk/nsApplicationAccessibleWrap.cpp --- mozilla.orig/accessible/src/atk/nsApplicationAccessibleWrap.cpp 2011-11-06 02:34:00.000000000 -0600 +++ mozilla/accessible/src/atk/nsApplicationAccessibleWrap.cpp 2011-11-26 20:32:59.939341632 -0600 @@ -52,19 +52,17 @@
#include <gtk/gtk.h> #include <atk/atk.h> +#include <dbus/dbus.h>
Including without checking if dbus is enabled/available? That might work in our build configuration but usually I'd like to have patches which could be applied and do the right thing for everyone.
-static bool sATKChecked = false; +static PRBool sATKChecked = PR_FALSE;
Why that? Starting with Firefox 10 PRBool is not used in the mozilla tree (minus nspr/nss) anymore intentionally.
+void +PreInit() +{ + static PRBool sChecked = FALSE;
PRBool again as in following places. Thanks for the work! I'm going to add the patch to the beta builds of Firefox 9 and Thunderbird 9 for now. Those are to be released December, 20th Or do you think we should release an official update based on FF8 and TB8 earlier? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.