Revision: 22062
Author:   [email protected]
Date:     Fri Jun 27 11:37:09 2014 UTC
Log: Remove remaining dependencies on allocation.h from a few platform files

I could move parts of allocation to base/ instead, but I see little to
no value in the allocation base classes, soo..

BUG=none
[email protected]
LOG=n

Review URL: https://codereview.chromium.org/351683009
http://code.google.com/p/v8/source/detail?r=22062

Modified:
 /branches/bleeding_edge/src/platform/time.h
 /branches/bleeding_edge/src/platform-posix.cc
 /branches/bleeding_edge/src/platform-win32.cc

=======================================
--- /branches/bleeding_edge/src/platform/time.h Tue Jun  3 08:12:43 2014 UTC
+++ /branches/bleeding_edge/src/platform/time.h Fri Jun 27 11:37:09 2014 UTC
@@ -8,7 +8,7 @@
 #include <time.h>
 #include <limits>

-#include "src/allocation.h"
+#include "src/base/macros.h"

 // Forward declarations.
 extern "C" {
@@ -30,7 +30,7 @@
 // This class represents a duration of time, internally represented in
 // microseonds.

-class TimeDelta V8_FINAL BASE_EMBEDDED {
+class TimeDelta V8_FINAL {
  public:
   TimeDelta() : delta_(0) {}

@@ -158,7 +158,7 @@
// This class represents an absolute point in time, internally represented as
 // microseconds (s/1,000,000) since 00:00:00 UTC, January 1, 1970.

-class Time V8_FINAL BASE_EMBEDDED {
+class Time V8_FINAL {
  public:
   static const int64_t kMillisecondsPerSecond = 1000;
   static const int64_t kMicrosecondsPerMillisecond = 1000;
@@ -295,7 +295,7 @@
 // Time::Now() may actually decrease or jump).  But note that TimeTicks may
 // "stand still", for example if the computer suspended.

-class TimeTicks V8_FINAL BASE_EMBEDDED {
+class TimeTicks V8_FINAL {
  public:
   TimeTicks() : ticks_(0) {}

=======================================
--- /branches/bleeding_edge/src/platform-posix.cc Fri Jun 27 08:38:56 2014 UTC +++ /branches/bleeding_edge/src/platform-posix.cc Fri Jun 27 11:37:09 2014 UTC
@@ -492,7 +492,7 @@
 // POSIX thread support.
 //

-class Thread::PlatformData : public Malloced {
+class Thread::PlatformData {
  public:
   PlatformData() : thread_(kNoThread) {}
   pthread_t thread_;  // Thread handle for pthread.
=======================================
--- /branches/bleeding_edge/src/platform-win32.cc Fri Jun 27 08:38:56 2014 UTC +++ /branches/bleeding_edge/src/platform-win32.cc Fri Jun 27 11:37:09 2014 UTC
@@ -1335,7 +1335,7 @@
 }


-class Thread::PlatformData : public Malloced {
+class Thread::PlatformData {
  public:
   explicit PlatformData(HANDLE thread) : thread_(thread) {}
   HANDLE thread_;

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to