From f4ede82c98052c99dd321e444c4586b132d86106 Mon Sep 17 00:00:00 2001 From: "jochen@chromium.org" Date: Fri, 27 Jun 2014 11:37:09 +0000 Subject: [PATCH] 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 R=bmeurer@chromium.org LOG=n Review URL: https://codereview.chromium.org/351683009 git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22062 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- src/platform-posix.cc | 2 +- src/platform-win32.cc | 2 +- src/platform/time.h | 8 ++++---- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/platform-posix.cc b/src/platform-posix.cc index 928edb8..bc9d8e2 100644 --- a/src/platform-posix.cc +++ b/src/platform-posix.cc @@ -492,7 +492,7 @@ void OS::StrNCpy(char* dest, int length, const char* src, size_t n) { // POSIX thread support. // -class Thread::PlatformData : public Malloced { +class Thread::PlatformData { public: PlatformData() : thread_(kNoThread) {} pthread_t thread_; // Thread handle for pthread. diff --git a/src/platform-win32.cc b/src/platform-win32.cc index 1897b75..f89c107 100644 --- a/src/platform-win32.cc +++ b/src/platform-win32.cc @@ -1335,7 +1335,7 @@ static unsigned int __stdcall ThreadEntry(void* arg) { } -class Thread::PlatformData : public Malloced { +class Thread::PlatformData { public: explicit PlatformData(HANDLE thread) : thread_(thread) {} HANDLE thread_; diff --git a/src/platform/time.h b/src/platform/time.h index 0cd234c..d21e85d 100644 --- a/src/platform/time.h +++ b/src/platform/time.h @@ -8,7 +8,7 @@ #include #include -#include "src/allocation.h" +#include "src/base/macros.h" // Forward declarations. extern "C" { @@ -30,7 +30,7 @@ class TimeTicks; // 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 @@ class TimeDelta V8_FINAL BASE_EMBEDDED { // 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 @@ inline Time operator+(const TimeDelta& delta, const Time& time) { // 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) {} -- 2.7.4