RESOLVED FIXED 20992
Build fails on GTK+ Mac OS
https://bugs.webkit.org/show_bug.cgi?id=20992
Summary Build fails on GTK+ Mac OS
Richard Hult
Reported 2008-09-22 08:54:46 PDT
JavaScriptCore/wtf/ThreadingGtk.cpp has this snippet at the beginning: #if !PLATFORM(DARWIN) static ThreadIdentifier mainThreadIdentifier; #endif Later in the same file the variable is used unconditionally, causing a build error when building with GTK+ on Mac OS. I'm not sure why it is conditionalized at all, my guess is that the right fix is to just remove the #if.
Attachments
remove platform ifdefs on ThreadingGtk (1.23 KB, patch)
2008-09-23 17:10 PDT, Jan Alonzo
alp: review+
Jan Alonzo
Comment 1 2008-09-23 17:10:42 PDT
Created attachment 23736 [details] remove platform ifdefs on ThreadingGtk Removed platform ifdef as suggested by Richard.
Eric Seidel (no email)
Comment 2 2008-09-23 19:58:32 PDT
Comment on attachment 23736 [details] remove platform ifdefs on ThreadingGtk There was a separate bug about this which I reviewed recently and had a different fix. I thought that had landed already. Are you sure you're up to date?
Eric Seidel (no email)
Comment 3 2008-09-23 19:59:22 PDT
Comment on attachment 23736 [details] remove platform ifdefs on ThreadingGtk Oh, nevermind. I think I was thinking of Qt. This code was probably copied from Qt, and probably doesn't need those #ifdefs.
Alp Toker
Comment 4 2008-09-23 20:40:49 PDT
Comment on attachment 23736 [details] remove platform ifdefs on ThreadingGtk r=me
Jan Alonzo
Comment 5 2008-09-24 00:45:21 PDT
landed in r36846
Note You need to log in before you can comment on or make changes to this bug.