Shashank S Mogaveera
b36143daa1
The prune_packed_objects() function uses the global the_repository variable. To help with the libification effort, change this function so it takes a 'struct repository' pointer as an argument instead. The function for_each_loose_file_in_source() uses a callback that only takes a single 'void *data' pointer. This means we cannot pass both the repository pointer and the 'opts' integer at the same time. To fix this, create a new 'struct prune_data' to hold both of them together, and update the prune_subdir() and prune_object() callbacks to read from this new struct. Update the builtins that call this function to pass their local `repo` parameters. As all of those builtins are declared with `RUN_SETUP` we know that they won't execute outside a repository and thus the repository parameter will be set. Unless the command was invoked with "-h" to print help, but all new uses of the `repo` variable are after a call to `parse_options()` which knows to exit early in case it sees "-h" Finally, drop the USE_THE_REPOSITORY_VARIABLE macro from prune-packed.c. Signed-off-by: Shashank S Mogaveera <shashu8660@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
12 lines
199 B
C
12 lines
199 B
C
#ifndef PRUNE_PACKED_H
|
|
#define PRUNE_PACKED_H
|
|
|
|
struct repository;
|
|
|
|
#define PRUNE_PACKED_DRY_RUN 01
|
|
#define PRUNE_PACKED_VERBOSE 02
|
|
|
|
void prune_packed_objects(struct repository *r, int opts);
|
|
|
|
#endif
|