Re: [GSOC PATCH 0/2] builtin/prune: remove dependency on global variables and 'the_repository'
- From
Ayush Chandekar <ayu.chandekar@gmail.com>
- Date
- Jun 25, 2025, 15:59 UTC
- Message-ID
- <CAE7as+aJS7qaqoRTZsiL87m_L8KMYe=EE=-uB40jh6yGXDKO_Q@mail.gmail.com>
- In-Reply-To
- <cover.1749343601.git.ayu.chandekar@gmail.com>
On Sun, Jun 8, 2025 at 6:37 AM Ayush Chandekar <ayu.chandekar@gmail.com> wrote:
Show 29 quoted lines
> > The aim of this patch series is to remove the definition '#define USE_THE_REPOSITORY_VARIABLE' > from "builtin/prune.c" by removing global variables and the global 'the_repository'. > > This patch series contains two patches: > > 1 - Move the global variable 'repository_format_precious_objects' into 'struct repository' > and update all affected code paths accordingly. > > 2 - Remove the dependency of 'the_repository' in "builtin/prunce.c", allowing the removal of > the definition. > > Ayush Chandekar (2): > repository: move 'repository_format_precious_objects' to repo scope > builtin/prune: stop depending on 'the_repository' > > builtin/gc.c | 2 +- > builtin/prune.c | 25 ++++++++++++------------- > builtin/repack.c | 2 +- > environment.c | 1 - > environment.h | 2 -- > repository.c | 1 + > repository.h | 1 + > setup.c | 5 ++++- > 8 files changed, 20 insertions(+), 19 deletions(-) > > -- > 2.49.0 >
This patch appears to have gone unnoticed as it hasn't seen any reviews yet. Resurfacing it in case it got overlooked.