Skip to content

perf(runner): drop round-2 fork leftovers from #761#797

Merged
Chemaclass merged 1 commit into
mainfrom
perf/761-fork-leftovers
Jul 14, 2026
Merged

perf(runner): drop round-2 fork leftovers from #761#797
Chemaclass merged 1 commit into
mainfrom
perf/761-fork-leftovers

Conversation

@Chemaclass

Copy link
Copy Markdown
Member

🤔 Background

Related #761

Two small per-run fork leftovers remained from the round-2 perf work: wait_for_job_slot shelled out to wc on every poll, and check_duplicate_functions ran a grep + awk + sort + uniq chain per test file.

💡 Changes

  • wait_for_job_slot counts running background jobs with a fork-free pure-bash line count over jobs -pr (via a return slot) instead of jobs -r | wc -l — 145 → 0 wc forks across a small parallel run
  • check_duplicate_functions folds its per-file grep + awk + sort + uniq scan into a single awk pass + sort; output and return codes are byte-identical
  • Adds unit tests for the job counter and a characterization test locking the sorted duplicate-name output; no behaviour change

wait_for_job_slot now counts running jobs via a fork-free line count over
jobs -pr (return slot) instead of jobs -r | wc -l on every poll (145 -> 0
wc forks across a small parallel run). check_duplicate_functions folds
its per-file grep + awk + sort + uniq into a single awk pass + sort.
No behaviour change; output and return codes are identical.
@Chemaclass Chemaclass added the enhancement New feature or request label Jul 14, 2026
@Chemaclass Chemaclass self-assigned this Jul 14, 2026
@Chemaclass Chemaclass merged commit 9fd06f4 into main Jul 14, 2026
37 checks passed
@Chemaclass Chemaclass deleted the perf/761-fork-leftovers branch July 14, 2026 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant