00a07bc1b7
This tries to remove includes of "-inl.h" headers from normal ".h" headers, thereby reducing the chance of any cyclic dependencies and decreasing the average size of our compilation units. Note that this change still leaves 7 violations of that rule in the code. However there now is the "tools/check-inline-includes.sh" tool detecting such violations. R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/1283033003 Cr-Commit-Position: refs/heads/master@{#30125}
20 lines
616 B
Bash
Executable File
20 lines
616 B
Bash
Executable File
#!/bin/bash
|
|
# Copyright 2015 the V8 project authors. All rights reserved.
|
|
# Use of this source code is governed by a BSD-style license that can be
|
|
# found in the LICENSE file.
|
|
|
|
v8_root=$(readlink -f $(dirname $BASH_SOURCE)/../)
|
|
headers=$(find "$v8_root/src" -name '*.h' -not -name '*-inl.h')
|
|
|
|
for header in $headers; do
|
|
inline_header_include=$(grep '#include ".*-inl.h"' "$header")
|
|
if [ -n "$inline_header_include" ]; then
|
|
echo "The following non-inline header seems to include an inline header:"
|
|
echo " Header : $header"
|
|
echo " Include: $inline_header_include"
|
|
echo
|
|
fi
|
|
done
|
|
|
|
echo "Kthxbye."
|