]> sourceware.org Git - valgrind.git/commitdiff
Minor tweaks for Python tooling updates.
authorNicholas Nethercote <n.nethercote@gmail.com>
Mon, 2 Oct 2023 21:11:42 +0000 (08:11 +1100)
committerNicholas Nethercote <n.nethercote@gmail.com>
Mon, 2 Oct 2023 21:44:15 +0000 (08:44 +1100)
Annoying that these are necessary, but that seems to be standard for
this stuff.

cachegrind/cg_annotate.in
cachegrind/cg_diff.in
cachegrind/cg_merge.in

index 4d9d3e32cdc7d83691ef99b1bb6a62698af5c314..fe3f411818e2a3b731b71f2e2225805329a8f9e9 100755 (executable)
@@ -50,6 +50,7 @@ def die(msg: str) -> NoReturn:
 
 SearchAndReplace = Callable[[str], str]
 
+
 # A typed wrapper for parsed args.
 class Args(Namespace):
     # None of these fields are modified after arg parsing finishes.
@@ -223,7 +224,7 @@ class Args(Namespace):
             p.print_usage(file=sys.stderr)
             die("argument --diff: requires exactly two Cachegrind output files")
 
-        return args0
+        return args0  # type: ignore [return-value]
 
 
 # Args are stored in a global for easy access.
@@ -318,6 +319,7 @@ class Events:
 # type is very hot and just using a type alias is much faster.
 Cc = list[int]
 
+
 # Add the counts in `a_cc` to `b_cc`.
 def add_cc_to_cc(a_cc: Cc, b_cc: Cc) -> None:
     for i, a_count in enumerate(a_cc):
@@ -375,6 +377,7 @@ def update_cc_extremes(self: Cc, min_cc: Cc, max_cc: Cc) -> None:
 #   `--mod-funcname`.
 # - Mname/mname: modified name, used for what could be an Mfl or an Mfn.
 
+
 # A deep cost centre with a dict for the inner mnames and CCs.
 class Dcc:
     outer_cc: Cc
@@ -881,7 +884,7 @@ def print_mname_summary(
     print()
 
     # Print LCCs.
-    threshold_mnames = set([])
+    threshold_mnames: set[str] = set([])
     cumul_cc = events.mk_empty_cc()
     for mname, lcc in list_mname_lcc:
         add_cc_to_cc(lcc.outer_cc, cumul_cc)
@@ -1090,9 +1093,7 @@ def print_annotated_src_files(
                     # mtimes as if they are all as early as the earliest one.
                     # Therefore, we warn only if the earliest source file is
                     # more recent than the cgout file.
-                    min_ofl_st_mtime_ns = min(
-                        [os.stat(ofl).st_mtime_ns for ofl in ofls]
-                    )
+                    min_ofl_st_mtime_ns = min(os.stat(ofl).st_mtime_ns for ofl in ofls)
 
                     for cgout_filename in args.cgout_filename:
                         if min_ofl_st_mtime_ns > os.stat(cgout_filename).st_mtime_ns:
@@ -1155,7 +1156,7 @@ def print_annotation_summary(
     print()
 
     total_cc = events.mk_empty_cc()
-    for (cc, label) in zip(annotated_ccs.ccs(), AnnotatedCcs.labels):
+    for cc, label in zip(annotated_ccs.ccs(), AnnotatedCcs.labels):
         printer.print_cc(cc, None, label)
         add_cc_to_cc(cc, total_cc)
 
index e193e99d78a99b64d6fc43a56af467c7f476a7a9..acc5484f1a7a9e7c883985de1938b2c0abde3a97 100755 (executable)
@@ -128,12 +128,13 @@ class Args(Namespace):
             help="file produced by Cachegrind",
         )
 
-        return p.parse_args(namespace=Args())
+        return p.parse_args(namespace=Args())  # type: ignore [return-value]
 
 
 # Args are stored in a global for easy access.
 args = Args.parse()
 
+
 # A single instance of this class is constructed, from `args` and the `events:`
 # line in the cgout file.
 class Events:
@@ -172,6 +173,7 @@ class Events:
 # type is very hot and just using a type alias is much faster.
 Cc = list[int]
 
+
 # Add the counts in `a_cc` to `b_cc`.
 def add_cc_to_cc(a_cc: Cc, b_cc: Cc) -> None:
     for i, a_count in enumerate(a_cc):
index 1201114ded0fcf26d4bc7762f8d025ab7657e483..7cc0f2b14b734585d46566a00ef7c63e4ba5c30b 100755 (executable)
@@ -74,12 +74,13 @@ class Args(Namespace):
             help="file produced by Cachegrind",
         )
 
-        return p.parse_args(namespace=Args())
+        return p.parse_args(namespace=Args())  # type: ignore [return-value]
 
 
 # Args are stored in a global for easy access.
 args = Args.parse()
 
+
 # A single instance of this class is constructed, from `args` and the `events:`
 # line in the cgout file.
 class Events:
This page took 0.042504 seconds and 5 git commands to generate.