Revision: 3074 Author: [email protected] Date: Thu Oct 15 08:01:36 2009 Log: Untangle some #include dependencies.
"jsregexp.h" and "jump-target.h" required "macro-assembler.h" to always be included first. Instead the include of "macro-assembler.h" has moved into those header files. "dateparser-inl.h" required "dateparser.h" to always be included first. Instead the include of "dateparser.h" has moved into "dateparser-inl.h". Review URL: http://codereview.chromium.org/267117 http://code.google.com/p/v8/source/detail?r=3074 Modified: /branches/bleeding_edge/src/ast.h /branches/bleeding_edge/src/dateparser-inl.h /branches/bleeding_edge/src/jsregexp.cc /branches/bleeding_edge/src/jsregexp.h /branches/bleeding_edge/src/jump-target.h /branches/bleeding_edge/src/objects-debug.cc /branches/bleeding_edge/src/runtime.cc ======================================= --- /branches/bleeding_edge/src/ast.h Tue Oct 13 02:37:17 2009 +++ /branches/bleeding_edge/src/ast.h Thu Oct 15 08:01:36 2009 @@ -30,12 +30,11 @@ #include "execution.h" #include "factory.h" +#include "jsregexp.h" +#include "jump-target.h" #include "runtime.h" #include "token.h" #include "variables.h" -#include "macro-assembler.h" -#include "jsregexp.h" -#include "jump-target.h" namespace v8 { namespace internal { ======================================= --- /branches/bleeding_edge/src/dateparser-inl.h Wed Sep 9 03:49:40 2009 +++ /branches/bleeding_edge/src/dateparser-inl.h Thu Oct 15 08:01:36 2009 @@ -28,6 +28,8 @@ #ifndef V8_DATEPARSER_INL_H_ #define V8_DATEPARSER_INL_H_ +#include "dateparser.h" + namespace v8 { namespace internal { ======================================= --- /branches/bleeding_edge/src/jsregexp.cc Wed Sep 2 00:34:51 2009 +++ /branches/bleeding_edge/src/jsregexp.cc Thu Oct 15 08:01:36 2009 @@ -45,13 +45,10 @@ #ifdef V8_NATIVE_REGEXP #if V8_TARGET_ARCH_IA32 -#include "ia32/macro-assembler-ia32.h" #include "ia32/regexp-macro-assembler-ia32.h" #elif V8_TARGET_ARCH_X64 -#include "x64/macro-assembler-x64.h" #include "x64/regexp-macro-assembler-x64.h" #elif V8_TARGET_ARCH_ARM -#include "arm/macro-assembler-arm.h" #include "arm/regexp-macro-assembler-arm.h" #else #error Unsupported target architecture. ======================================= --- /branches/bleeding_edge/src/jsregexp.h Wed Jul 29 01:10:19 2009 +++ /branches/bleeding_edge/src/jsregexp.h Thu Oct 15 08:01:36 2009 @@ -28,6 +28,8 @@ #ifndef V8_JSREGEXP_H_ #define V8_JSREGEXP_H_ +#include "macro-assembler.h" + namespace v8 { namespace internal { ======================================= --- /branches/bleeding_edge/src/jump-target.h Wed Sep 9 03:49:40 2009 +++ /branches/bleeding_edge/src/jump-target.h Thu Oct 15 08:01:36 2009 @@ -28,6 +28,8 @@ #ifndef V8_JUMP_TARGET_H_ #define V8_JUMP_TARGET_H_ +#include "macro-assembler.h" + namespace v8 { namespace internal { ======================================= --- /branches/bleeding_edge/src/objects-debug.cc Mon Sep 21 03:35:47 2009 +++ /branches/bleeding_edge/src/objects-debug.cc Thu Oct 15 08:01:36 2009 @@ -29,7 +29,6 @@ #include "disassembler.h" #include "disasm.h" -#include "macro-assembler.h" #include "jsregexp.h" namespace v8 { ======================================= --- /branches/bleeding_edge/src/runtime.cc Tue Oct 13 01:13:45 2009 +++ /branches/bleeding_edge/src/runtime.cc Thu Oct 15 08:01:36 2009 @@ -34,18 +34,17 @@ #include "arguments.h" #include "compiler.h" #include "cpu.h" -#include "dateparser.h" #include "dateparser-inl.h" #include "debug.h" #include "execution.h" #include "jsregexp.h" +#include "parser.h" #include "platform.h" #include "runtime.h" #include "scopeinfo.h" -#include "v8threads.h" #include "smart-pointer.h" -#include "parser.h" #include "stub-cache.h" +#include "v8threads.h" namespace v8 { namespace internal { --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
