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

Unified 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 side-by-side diff with in-line comments
Download patch
« webrtc/tools/author_line_count ('K') | « webrtc/tools/author_line_count ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/tools/header_usage
diff --git a/webrtc/tools/header_usage b/webrtc/tools/header_usage
new file mode 100755
index 0000000000000000000000000000000000000000..18556ac6e0a674eb7caee84bafd70bea7df4e8a6
--- /dev/null
+++ b/webrtc/tools/header_usage
@@ -0,0 +1,15 @@
+#! /bin/bash
+
+# This script is run in a git repository. It lists all header files,
+# sorted by the number of other files where the file name of the file
+# occurs. It is intentionally not limited to only source files, and
+# there may be some false hits because we search only for the file
+# part (sans directory).
+#
+# Headers close to the top of the list are candidates for removal.
+
+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.
+ 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.
+ count="$(git grep -l -F "$name" | wc -l)"
+ echo "$count" "$header"
+done | sort -n
« 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