Reviewers: ,

Message:
Patch by Michael Lutz <[email protected]>.

LGTM.

Description:
Port r10490 and r10615 to cygwin.


BUG=
TEST=


Please review this at http://codereview.chromium.org/9423041/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M src/platform-cygwin.cc


Index: src/platform-cygwin.cc
diff --git a/src/platform-cygwin.cc b/src/platform-cygwin.cc
index c27e3c982f6540ddfe729496c2a27bf5593bae58..79134da35255b529e95920fc6542116fe9402dba 100644
--- a/src/platform-cygwin.cc
+++ b/src/platform-cygwin.cc
@@ -365,16 +365,9 @@ class Thread::PlatformData : public Malloced {


 Thread::Thread(const Options& options)
-    : data_(new PlatformData),
-      stack_size_(options.stack_size) {
-  set_name(options.name);
-}
-
-
-Thread::Thread(const char* name)
-    : data_(new PlatformData),
-      stack_size_(0) {
-  set_name(name);
+    : data_(new PlatformData()),
+      stack_size_(options.stack_size()) {
+  set_name(options.name());
 }


@@ -617,8 +610,10 @@ class Sampler::PlatformData : public Malloced {

 class SamplerThread : public Thread {
  public:
+  static const int kSamplerThreadStackSize = 64 * KB;
+
   explicit SamplerThread(int interval)
-      : Thread("SamplerThread"),
+      : Thread(Thread::Options("SamplerThread", kSamplerThreadStackSize)),
         interval_(interval) {}

   static void AddActiveSampler(Sampler* sampler) {


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to