git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [GSoC PATCH] environment.h: remove unused variables

From
Patrick Steinhardt <ps@pks.im>
Date
Apr 7, 2025, 08:03 UTC
Message-ID
<Z_OGyqVHaUkFIh1T@pks.im>
In-Reply-To
<2c547567-2b72-476c-9fc5-71cac050fa15@gmail.com>
On Sat, Apr 05, 2025 at 10:15:32PM +0530, Arnav Bhate wrote:
Show 26 quoted lines
> packed_git_window_size and packed_git_limit are not used anywhere in
> the codebase. A search found that all references were removed in
> d284713bae (config: make `packed_git_(limit|window_size)` non-global
> variables, 2024-12-03), except the ones in this file, as they were moved
> to struct repo_settings.
> 
> Remove packed_git_window_size and packed_git_limit from environment.h.
> 
> Signed-off-by: Arnav Bhate <bhatearnav@gmail.com>
> ---
>  environment.h | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/environment.h b/environment.h
> index 45e690f203..cf6fc04e70 100644
> --- a/environment.h
> +++ b/environment.h
> @@ -152,8 +152,6 @@ extern char *apply_default_ignorewhitespace;
>  extern char *git_attributes_file;
>  extern int zlib_compression_level;
>  extern int pack_compression_level;
> -extern size_t packed_git_window_size;
> -extern size_t packed_git_limit;
>  extern unsigned long big_file_threshold;
>  extern unsigned long pack_size_limit_cfg;
>  extern int max_allowed_tree_depth;
Good catch indeed, the patch looks obviously good to me!
Patrick
Previous: Arnav Bhate
Message 2 of 2 in “environment.h: remove unused variables”
  1. environment.h: remove unused variablesArnav Bhate, Apr 5, 2025
  2. Patrick SteinhardtApr 7, 2025

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.