diff options
| author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-21 21:47:58 +0000 |
|---|---|---|
| committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-21 21:47:58 +0000 |
| commit | a425c0d7d5acb3f1a47eef0639e436ff60522ba5 (patch) | |
| tree | 6e7bc75a9abab4756bea0e6b5a18dab33fa2927b /third_party/yasm | |
| parent | 893f78d19eb3f1087c013c8d4ee00df053945668 (diff) | |
| download | chromium_src-a425c0d7d5acb3f1a47eef0639e436ff60522ba5.zip chromium_src-a425c0d7d5acb3f1a47eef0639e436ff60522ba5.tar.gz chromium_src-a425c0d7d5acb3f1a47eef0639e436ff60522ba5.tar.bz2 | |
Check for unused variables in GN.
A check for unused variables was missing for some helper functions.
The main thing that this affects is the "config" function, where
people were confused when setting deps or configs didn't work the
way they expected (because they were unused).
BUG=392026
R=dpranke@chromium.org
Review URL: https://codereview.chromium.org/399143006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284509 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'third_party/yasm')
| -rw-r--r-- | third_party/yasm/BUILD.gn | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/third_party/yasm/BUILD.gn b/third_party/yasm/BUILD.gn index 4e0c979..3463852 100644 --- a/third_party/yasm/BUILD.gn +++ b/third_party/yasm/BUILD.gn @@ -41,11 +41,6 @@ if (current_toolchain == host_toolchain) { "source/config/$os", "source/patched-yasm", ] - inputs = [ - config_makefile, - "source/config/$os/config.h", - "source/config/$os/libyasm-stdint.h", - ] defines = [ "HAVE_CONFIG_H" ] if (is_posix) { cflags = [ "-std=gnu99" ] @@ -380,6 +375,7 @@ if (current_toolchain == host_toolchain) { compiled_action("generate_module") { tool = ":genmodule" + inputs = [ config_makefile ] sources = [ "source/patched-yasm/libyasm/module.in" ] outputs = [ "$target_gen_dir/module.c" ] args = [ |