smallsolar/u-boot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

codman: Sort files alphabetically and drop inactive column

Browse Source 
Files appear in arbitrary order, making it hard to find specific
files. Sort them alphabetically by filename for easier navigation.

Also remove the inactive lines column from file output to reduce
clutter, keeping only: filename, total lines, active lines, and
percentage active.

Co-developed-by: Claude <noreply@anthropic.com>
Signed-off-by: Simon Glass <simon.glass@canonical.com>
This commit is contained in:
Simon Glass
2025-12-08 17:08:30 -07:00
parent 0406a0d829
commit e0543235e1
1 changed files with 3 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
tools/codman/output.py
View File

@@ -379,13 +379,8 @@ def print_dir_stats(dir_stats, file_results, by_subdirs, show_files,
# Show individual files if requested
if show_files and stats.files:
# Sort files by inactive lines (descending) for line-level, or
# alphabetically otherwise
if file_results:
sorted_files = sorted(stats.files, key=lambda x: x['inactive'],
reverse=True)
else:
sorted_files = sorted(stats.files, key=lambda x: x['path'])
# Sort files alphabetically by filename
sorted_files = sorted(stats.files, key=lambda x: os.path.basename(x['path']))
for info in sorted_files:
# Skip files with 0 active lines unless show_empty is set
@@ -400,8 +395,7 @@ def print_dir_stats(dir_stats, file_results, by_subdirs, show_files,
# Show line-level details
pct_active = percent(info['active'], info['total'])
print(f" {filename:<38} {info['total']:>7} "
f"{info['active']:>7} {pct_active:>6.1f} "
f"{info['inactive']:>7}")
f"{info['active']:>7} {pct_active:>6.1f}")
else:
# Show file-level only
print(f" {filename:<38} {info['total']:>7} lines")