[PATCH 0/3] revision: refactor ref_excludes to ref_visibility
- From
John Cai via GitGitGadget <gitgitgadget@gmail.com>
- Date
- Jun 21, 2023, 19:35 UTC
- Message-ID
- <pull.1515.git.git.1687376112.gitgitgadget@gmail.com>
The ref_excludes API is used to tell which refs should be excluded. However, there are times when we would want to add refs to explicitly include as well. 4fe42f326e (pack-refs: teach pack-refs --include option, 2023-05-12) taught pack-refs how to include certain refs, but did it in a more manual way by keeping the ref patterns in a separate string list. Instead, we can easily extend the ref_excludes API to include refs as well, since this use case fits into the API nicely.
Refactor the API by renaming it to ref_visibility, and add a ref_visible() helper that takes into account ref inclusion.
John Cai (3): revision: rename ref_excludes to ref_visibility revision: add ref_visible() helper pack-refs: use new ref_visible() helper
builtin/pack-refs.c | 20 ++++---- builtin/rev-parse.c | 18 +++---- refs.h | 2 +- refs/files-backend.c | 11 +---- revision.c | 98 +++++++++++++++++++++++---------------- revision.h | 37 +++++++++++---- t/helper/test-ref-store.c | 8 ++-- 7 files changed, 111 insertions(+), 83 deletions(-)
base-commit: 6640c2d06d112675426cf436f0594f0e8c614848 Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1515%2Fjohn-cai%2Fjc%2Frefactor-ref-excludes-v1 Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1515/john-cai/jc/refactor-ref-excludes-v1 Pull-Request: https://github.com/git/git/pull/1515
-- gitgitgadget