Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(759)

Side by Side Diff: webrtc/tools/header_usage.sh

Issue 2363423002: header_usage.sh script: Exclude matches in gyp and gn files. (Closed)
Patch Set: Ignore all *.gn* and *.gyp* files. Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/bin/bash 1 #!/bin/bash
2 2
3 # Copyright (c) 2016 The WebRTC project authors. All Rights Reserved. 3 # Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
4 # 4 #
5 # Use of this source code is governed by a BSD-style license 5 # Use of this source code is governed by a BSD-style license
6 # that can be found in the LICENSE file in the root of the source 6 # that can be found in the LICENSE file in the root of the source
7 # tree. An additional intellectual property rights grant can be found 7 # tree. An additional intellectual property rights grant can be found
8 # in the file PATENTS. All contributing project authors may 8 # in the file PATENTS. All contributing project authors may
9 # be found in the AUTHORS file in the root of the source tree. 9 # be found in the AUTHORS file in the root of the source tree.
10 10
11 # This script is run in a git repository. It lists all header files, 11 # This script is run in a git repository. It lists all header files,
12 # sorted by the number of other files where the file name of the file 12 # sorted by the number of other files where the file name of the file
13 # occurs. It is intentionally not limited to only source files, and 13 # occurs. It is intentionally not limited to only source files, and
14 # there may be some false hits because we search only for the file 14 # there may be some false hits because we search only for the file
15 # part (sans directory). It is quite slow. 15 # part (sans directory). It is quite slow.
16 # 16 #
17 # Headers close to the top of the list are candidates for removal. 17 # Headers close to the top of the list are candidates for removal.
18 18
19 git ls-files '*.h' '*.hpp' | while read header ; do 19 git ls-files '*.h' '*.hpp' | while read header ; do
20 name="$(basename "${header}")" 20 name="$(basename "${header}")"
21 count="$(git grep -l -F "${name}" | wc -l)" 21 count="$(git grep -l -F "${name}" \
22 | grep -v -e '\.gn' -e '\.gyp' \
23 | wc -l)"
22 echo "${count}" "${header}" 24 echo "${count}" "${header}"
23 done | sort -n 25 done | sort -n
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698