Title: [105290] trunk/Source/_javascript_Core
Revision
105290
Author
[email protected]
Date
2012-01-18 10:10:30 -0800 (Wed, 18 Jan 2012)

Log Message

Cross-platform processor core counter fix
https://bugs.webkit.org/show_bug.cgi?id=76540

Patch by Roland Takacs <[email protected]> on 2012-01-18
Reviewed by Zoltan Herczeg.

I attached "OS(FREEBSD)" to "#if OS(DARWIN) || OS(OPENBSD) || OS(NETBSD)"
and I removed the OS checking macros from ParallelJobsGeneric.cpp because
the NumberOfCores.cpp contains them for counting CPU cores.
The processor core counter patch located at
https://bugs.webkit.org/show_bug.cgi?id=76530

* wtf/NumberOfCores.cpp:
* wtf/ParallelJobsGeneric.cpp:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (105289 => 105290)


--- trunk/Source/_javascript_Core/ChangeLog	2012-01-18 18:06:55 UTC (rev 105289)
+++ trunk/Source/_javascript_Core/ChangeLog	2012-01-18 18:10:30 UTC (rev 105290)
@@ -1,3 +1,19 @@
+2012-01-18  Roland Takacs  <[email protected]>
+
+        Cross-platform processor core counter fix
+        https://bugs.webkit.org/show_bug.cgi?id=76540
+
+        Reviewed by Zoltan Herczeg.
+
+        I attached "OS(FREEBSD)" to "#if OS(DARWIN) || OS(OPENBSD) || OS(NETBSD)"
+        and I removed the OS checking macros from ParallelJobsGeneric.cpp because
+        the NumberOfCores.cpp contains them for counting CPU cores.
+        The processor core counter patch located at
+        https://bugs.webkit.org/show_bug.cgi?id=76530
+
+        * wtf/NumberOfCores.cpp:
+        * wtf/ParallelJobsGeneric.cpp:
+
 2012-01-18  Csaba Osztrogonác  <[email protected]>
 
         Cross-platform processor core counter

Modified: trunk/Source/_javascript_Core/wtf/NumberOfCores.cpp (105289 => 105290)


--- trunk/Source/_javascript_Core/wtf/NumberOfCores.cpp	2012-01-18 18:06:55 UTC (rev 105289)
+++ trunk/Source/_javascript_Core/wtf/NumberOfCores.cpp	2012-01-18 18:10:30 UTC (rev 105290)
@@ -26,7 +26,7 @@
 #include "config.h"
 #include "NumberOfCores.h"
 
-#if OS(DARWIN) || OS(OPENBSD) || OS(NETBSD)
+#if OS(DARWIN) || OS(OPENBSD) || OS(NETBSD) || OS(FREEBSD)
 #include <sys/sysctl.h>
 #include <sys/types.h>
 #elif OS(LINUX) || OS(AIX) || OS(SOLARIS)

Modified: trunk/Source/_javascript_Core/wtf/ParallelJobsGeneric.cpp (105289 => 105290)


--- trunk/Source/_javascript_Core/wtf/ParallelJobsGeneric.cpp	2012-01-18 18:06:55 UTC (rev 105289)
+++ trunk/Source/_javascript_Core/wtf/ParallelJobsGeneric.cpp	2012-01-18 18:10:30 UTC (rev 105290)
@@ -30,18 +30,8 @@
 #if ENABLE(THREADING_GENERIC)
 
 #include "ParallelJobs.h"
-#include "UnusedParam.h"
 #include <wtf/NumberOfCores.h>
 
-#if OS(DARWIN) || OS(OPENBSD) || OS(NETBSD)
-#include <sys/sysctl.h>
-#include <sys/types.h>
-#elif OS(LINUX) || OS(AIX) || OS(SOLARIS)
-#include <unistd.h>
-#elif OS(WINDOWS)
-#include <windows.h>
-#endif
-
 namespace WTF {
 
 Vector< RefPtr<ParallelEnvironment::ThreadPrivate> >* ParallelEnvironment::s_threadPool = 0;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to