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

Side by Side Diff: webrtc/tools/header_usage

Issue 2061683002: New misc scripts, header_usage.sh and author_line_count.sh. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 6 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
OLDNEW
(Empty)
1 #! /bin/bash
2
3 # This script is run in a git repository. It lists all header files,
4 # sorted by the number of other files where the file name of the file
5 # occurs. It is intentionally not limited to only source files, and
6 # there may be some false hits because we search only for the file
7 # part (sans directory).
8 #
9 # Headers close to the top of the list are candidates for removal.
10
11 git ls-files '*.h' | while read header ; do
kjellander_webrtc 2016/06/13 12:49:52 We do have two .hpp files. Might want to rename th
nisse-webrtc 2016/06/14 09:38:47 Added '*.hpp' to the ls-files argument list.
12 name="$(basename "$header")"
kjellander_webrtc 2016/06/13 12:49:52 Prefer curly brace quoting of variables: https://e
nisse-webrtc 2016/06/14 09:38:47 Done.
13 count="$(git grep -l -F "$name" | wc -l)"
14 echo "$count" "$header"
15 done | sort -n
OLDNEW
« webrtc/tools/author_line_count ('K') | « webrtc/tools/author_line_count ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698