Include what you use for the files zone*

src/zone-inl.h

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 10 matching lines @@
 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 #ifndef V8_ZONE_INL_H_
 #define V8_ZONE_INL_H_
 
+#include <stddef.h>

[Kevin Millikin (Chromium), 2012/01/30 09:25:45]

<stddef.h> and <stdint.h> come to us via ../include/v8stdint.h, which comes to
us via globals.h.  I think it's safe to leave them out of individual files
(that's the whole point of globals.h).  Since they can't participate in a
circular dependency, I'm not much worried about having them explicit everywhere.

+#include <stdint.h>
+
+#include "checks.h"

[Kevin Millikin (Chromium), 2012/01/30 09:25:45]

Since zone.h includes checks.h, globals.h, and splay-tree.h, and this file
includes zone.h, and they are part of the same 'module', you can feel free to
leave them out here (or put them in, I don't care too much).

+#include "counters.h"
+#include "globals.h"
 #include "isolate.h"
+#include "splay-tree.h"
+#include "utils.h"
+#include "v8-counters.h"
 #include "zone.h"

[Kevin Millikin (Chromium), 2012/01/30 09:25:45]

Please include zone.h first, separated from the other includes by a blank line,
like so:

#include "zone.h"

#include "counters.h"
etc.

-#include "v8-counters.h"
 
 namespace v8 {
 namespace internal {
 
 
 AssertNoZoneAllocation::AssertNoZoneAllocation()
     : prev_(Isolate::Current()->zone_allow_allocation()) {
   Isolate::Current()->set_zone_allow_allocation(false);
 }
 
@@ skipping 100 matching lines @@
 
 
 int ZoneScope::nesting() {
   return Isolate::Current()->zone()->scope_nesting_;
 }
 
 
 } }  // namespace v8::internal
 
 #endif  // V8_ZONE_INL_H_