Skip to content

Clean cleanups

Elijah Newren via GitGitGadget git 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest4 patches 2 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4dir: fix a few confusing commentsdir.c+6 -50 replies
2/4dir, clean: avoid disallowed behaviorbuiltin/clean.c dir.c+5 -10 replies
3/4clean: consolidate handling of ignored parametersbuiltin/clean.c+7 -90 replies
4/4clean: optimize and document cases where we recurse into subdirectoriesbuiltin/clean.c+31 -22 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/4] Clean cleanups

This fixes the performance regression reported at [1], and also performs a few other cleanups found while investigating the issue.

[1]
https://lore.kernel.org/git/CAJB88a23uU2WfB0mnB9NfNbtgmABhNOWNOEMBt7rRVu7uL_C9A@mail.gmail.com/

Elijah Newren (4):
dir: fix a few confusing comments
dir, clean: avoid disallowed behavior
clean: consolidate handling of ignored parameters
clean: optimize and document cases where we recurse into
subdirectories

 builtin/clean.c | 49 ++++++++++++++++++++++++++++++++++++++-----------
 dir.c           | 15 ++++++++++-----
 2 files changed, 48 insertions(+), 16 deletions(-)

base-commit: 20514004ddf1a3528de8933bc32f284e175e1012
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-806%2Fnewren%2Fclean-cleanups-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-806/newren/clean-cleanups-v1
Pull-Request: https://github.com/git/git/pull/806

Signature · 1 line
gitgitgadget

Read the cover letter in its thread