[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [tor/master] practracker: make filename terminology uniform



commit e3f7e5e65ed29a4a6e210ab50a731bba6e85f31c
Author: Nick Mathewson <nickm@xxxxxxxxxxxxxx>
Date:   Thu Aug 29 09:20:27 2019 -0400

    practracker: make filename terminology uniform
    
    Previously we often referred to "C files" and "H files", which is
    more ambiguous than ".c files" and ".h files".
---
 scripts/maint/practracker/README         |  8 ++++----
 scripts/maint/practracker/includes.py    |  2 +-
 scripts/maint/practracker/practracker.py | 16 ++++++++--------
 3 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/scripts/maint/practracker/README b/scripts/maint/practracker/README
index d2ad8220c..d978b3980 100644
--- a/scripts/maint/practracker/README
+++ b/scripts/maint/practracker/README
@@ -5,10 +5,10 @@ get worse.
 Right now, practracker looks for the following kinds of
 best-practices violations:
 
-  C files greater than 3000 lines long
-  H files greater than 500 lines long
-  C files with more than 50 includes
-  H files with more than 15 includes
+  .c files greater than 3000 lines long
+  .h files greater than 500 lines long
+  .c files with more than 50 includes
+  .h files with more than 15 includes
 
   All files that include a local header not listed in a .may_include
   file in the same directory, when that .may_include file has an
diff --git a/scripts/maint/practracker/includes.py b/scripts/maint/practracker/includes.py
index fcc527c98..397439b4e 100755
--- a/scripts/maint/practracker/includes.py
+++ b/scripts/maint/practracker/includes.py
@@ -212,7 +212,7 @@ def consider_include_rules(fname, f):
     log_sorted_levels = False
 
 def walk_c_files(topdir="src"):
-    """Run through all c and h files under topdir, looking for
+    """Run through all .c and .h files under topdir, looking for
        include-rule violations. Yield those violations."""
 
     for dirpath, dirnames, fnames in os.walk(topdir):
diff --git a/scripts/maint/practracker/practracker.py b/scripts/maint/practracker/practracker.py
index 73e4cb3e8..b477cd60c 100755
--- a/scripts/maint/practracker/practracker.py
+++ b/scripts/maint/practracker/practracker.py
@@ -8,7 +8,7 @@ violate some of our best practices and they are not found in the optional
 exceptions file, then log a problem about them.
 
 We currently do metrics about file size, function size and number of includes,
-for C files and headers.
+for C source files and headers.
 
 practracker.py should be run with its second argument pointing to the Tor
 top-level source directory like this:
@@ -144,10 +144,10 @@ HEADER="""\
 #
 # There are three kinds of problems that we recognize right now:
 #   function-size -- a function of more than {MAX_FUNCTION_SIZE} lines.
-#   file-size -- a C file of more than {MAX_FILE_SIZE} lines, or an H
+#   file-size -- a .c file of more than {MAX_FILE_SIZE} lines, or a .h
 #      file with more than {MAX_H_FILE_SIZE} lines.
-#   include-count -- a C file with more than {MAX_INCLUDE_COUNT} #includes,
-       or an H file with more than {MAX_H_INCLUDE_COUNT} #includes.
+#   include-count -- a .c file with more than {MAX_INCLUDE_COUNT} #includes,
+       or a .h file with more than {MAX_H_INCLUDE_COUNT} #includes.
 #   dependency-violation -- a file includes a header that it should
 #      not, according to an advisory .may_include file.
 #
@@ -189,13 +189,13 @@ def main(argv):
     parser.add_argument("--terse", action="store_true",
                         help="Do not emit helpful instructions.")
     parser.add_argument("--max-h-file-size", default=MAX_H_FILE_SIZE,
-                        help="Maximum lines per .H file")
+                        help="Maximum lines per .h file")
     parser.add_argument("--max-h-include-count", default=MAX_H_INCLUDE_COUNT,
-                        help="Maximum includes per .H file")
+                        help="Maximum includes per .h file")
     parser.add_argument("--max-file-size", default=MAX_FILE_SIZE,
-                        help="Maximum lines per C file")
+                        help="Maximum lines per .c file")
     parser.add_argument("--max-include-count", default=MAX_INCLUDE_COUNT,
-                        help="Maximum includes per C file")
+                        help="Maximum includes per .c file")
     parser.add_argument("--max-function-size", default=MAX_FUNCTION_SIZE,
                         help="Maximum lines per function")
     parser.add_argument("--max-dependency-violations", default=MAX_DEP_VIOLATIONS,



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits