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

Re: [PATCH] config.h: remove unused git_configset_add_parameters()

From
Glen Choo <chooglen@google.com>
Date
Feb 7, 2023, 17:55 UTC
Message-ID
<kl6lo7q5icmr.fsf@chooglen-macbookpro.roam.corp.google.com>
In-Reply-To
<patch-1.1-31f4a08c068-20230207T161453Z-avarab@gmail.com>
Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
> This function was removed in ecec57b3c97 (config: respect includes in
> protected config, 2022-10-13), but its prototype was left here.
Ugh, that's my mistake. Thanks for the catch.
Show 22 quoted lines
> diff --git a/config.h b/config.h
> index ef9eade6414..7606246531a 100644
> --- a/config.h
> +++ b/config.h
> @@ -447,15 +447,6 @@ void git_configset_init(struct config_set *cs);
>   */
>  int git_configset_add_file(struct config_set *cs, const char *filename);
>  
> -/**
> - * Parses command line options and environment variables, and adds the
> - * variable-value pairs to the `config_set`. Returns 0 on success, or -1
> - * if there is an error in parsing. The caller decides whether to free
> - * the incomplete configset or continue using it when the function
> - * returns -1.
> - */
> -int git_configset_add_parameters(struct config_set *cs);
> -
>  /**
>   * Finds and returns the value list, sorted in order of increasing priority
>   * for the configuration variable `key` and config set `cs`. When the
> -- 
> 2.39.1.1430.gb2471c0aaf4
Reviewed-by: Glen Choo <chooglen@google.com>
Previous: Ævar Arnfjörð Bjarmason
Message 2 of 2 in “config.h: remove unused git_configset_add_parameters()”
  1. config.h: remove unused git_configset_add_parameters()Ævar Arnfjörð Bjarmason, Feb 7, 2023
  2. Glen ChooFeb 7, 2023

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.