undef/git-commit-vandalism
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

config.c: plumb config_source through static fns

Browse Source 
This reduces the direct dependence on the global "struct config_source",
which will make it easier to remove in a later commit.

To minimize the changes we need to make, we rename the current variable
from "cf" to "cf_global", and the plumbed arg uses the old name "cf".
This is a little unfortunate, since we now have the confusingly named
"struct config_source cf" everywhere (which is a holdover from before
4d8dd1494e (config: make parsing stack struct independent from actual
data source, 2013-07-12), when the struct used to be called
"config_file"), but we will rename "cf" to "cs" by the end of the
series.

In some cases (public functions and config callback functions), there
isn't an obvious way to plumb "struct config_source" through function
args. As a workaround, add references to "cf_global" that we'll address
in later commits.

The remaining references to "cf_global" are direct assignments to
"cf_global", which we'll also address in a later commit.

Signed-off-by: Glen Choo <chooglen@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
Glen Choo 2023-03-28 17:51:47 +00:00 committed by Junio C Hamano
parent d15644fe02
commit c97f3ed256
1 changed files with 84 additions and 69 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

153
config.c
View File

@ -54,8 +54,8 @@ struct config_source {
* These variables record the "current" config source, which * These variables record the "current" config source, which
* can be accessed by parsing callbacks. * can be accessed by parsing callbacks.
* *
* The "cf" variable will be non-NULL only when we are actually parsing a real * The "cf_global" variable will be non-NULL only when we are actually
* config source (file, blob, cmdline, etc). * parsing a real config source (file, blob, cmdline, etc).
* *
* The "current_config_kvi" variable will be non-NULL only when we are feeding * The "current_config_kvi" variable will be non-NULL only when we are feeding
* cached config from a configset into a callback. * cached config from a configset into a callback.
@ -66,15 +66,16 @@ struct config_source {
* or it's a function which can be reused for non-config purposes, and should * or it's a function which can be reused for non-config purposes, and should
* fall back to some sane behavior). * fall back to some sane behavior).
*/ */
static struct config_source *cf; static struct config_source *cf_global;
static struct key_value_info *current_config_kvi; static struct key_value_info *current_config_kvi;
/* /*
* Similar to the variables above, this gives access to the "scope" of the * Similar to the variables above, this gives access to the "scope" of the
* current value (repo, global, etc). For cached values, it can be found via * current value (repo, global, etc). For cached values, it can be found via
* the current_config_kvi as above. During parsing, the current value can be * the current_config_kvi as above. During parsing, the current value can be
* found in this variable. It's not part of "cf" because it transcends a single * found in this variable. It's not part of "cf_global" because it transcends a
* file (i.e., a file included from .git/config is still in "repo" scope). * single file (i.e., a file included from .git/config is still in "repo"
* scope).
*/ */
static enum config_scope current_parsing_scope; static enum config_scope current_parsing_scope;
@ -156,7 +157,8 @@ static const char include_depth_advice[] = N_(
"from\n" "from\n"
" %s\n" " %s\n"
"This might be due to circular includes."); "This might be due to circular includes.");
static int handle_path_include(const char *path, struct config_include_data *inc) static int handle_path_include(struct config_source *cf, const char *path,
struct config_include_data *inc)
{ {
int ret = 0; int ret = 0;
struct strbuf buf = STRBUF_INIT; struct strbuf buf = STRBUF_INIT;
@ -210,7 +212,8 @@ static void add_trailing_starstar_for_dir(struct strbuf *pat)
strbuf_addstr(pat, "**"); strbuf_addstr(pat, "**");
} }
static int prepare_include_condition_pattern(struct strbuf *pat) static int prepare_include_condition_pattern(struct config_source *cf,
struct strbuf *pat)
{ {
struct strbuf path = STRBUF_INIT; struct strbuf path = STRBUF_INIT;
char *expanded; char *expanded;
@ -245,7 +248,8 @@ static int prepare_include_condition_pattern(struct strbuf *pat)
return prefix; return prefix;
} }
static int include_by_gitdir(const struct config_options *opts, static int include_by_gitdir(struct config_source *cf,
const struct config_options *opts,
const char *cond, size_t cond_len, int icase) const char *cond, size_t cond_len, int icase)
{ {
struct strbuf text = STRBUF_INIT; struct strbuf text = STRBUF_INIT;
@ -261,7 +265,7 @@ static int include_by_gitdir(const struct config_options *opts,
strbuf_realpath(&text, git_dir, 1); strbuf_realpath(&text, git_dir, 1);
strbuf_add(&pattern, cond, cond_len); strbuf_add(&pattern, cond, cond_len);
prefix = prepare_include_condition_pattern(&pattern); prefix = prepare_include_condition_pattern(cf, &pattern);
again: again:
if (prefix < 0) if (prefix < 0)
@ -342,14 +346,14 @@ static void populate_remote_urls(struct config_include_data *inc)
{ {
struct config_options opts; struct config_options opts;
struct config_source *store_cf = cf; struct config_source *store_cf = cf_global;
struct key_value_info *store_kvi = current_config_kvi; struct key_value_info *store_kvi = current_config_kvi;
enum config_scope store_scope = current_parsing_scope; enum config_scope store_scope = current_parsing_scope;
opts = *inc->opts; opts = *inc->opts;
opts.unconditional_remote_url = 1; opts.unconditional_remote_url = 1;
cf = NULL; cf_global = NULL;
current_config_kvi = NULL; current_config_kvi = NULL;
current_parsing_scope = 0; current_parsing_scope = 0;
@ -357,7 +361,7 @@ static void populate_remote_urls(struct config_include_data *inc)
string_list_init_dup(inc->remote_urls); string_list_init_dup(inc->remote_urls);
config_with_options(add_remote_url, inc->remote_urls, inc->config_source, &opts); config_with_options(add_remote_url, inc->remote_urls, inc->config_source, &opts);
cf = store_cf; cf_global = store_cf;
current_config_kvi = store_kvi; current_config_kvi = store_kvi;
current_parsing_scope = store_scope; current_parsing_scope = store_scope;
} }
@ -406,15 +410,16 @@ static int include_by_remote_url(struct config_include_data *inc,
inc->remote_urls); inc->remote_urls);
} }
static int include_condition_is_true(struct config_include_data *inc, static int include_condition_is_true(struct config_source *cf,
struct config_include_data *inc,
const char *cond, size_t cond_len) const char *cond, size_t cond_len)
{ {
const struct config_options *opts = inc->opts; const struct config_options *opts = inc->opts;
if (skip_prefix_mem(cond, cond_len, "gitdir:", &cond, &cond_len)) if (skip_prefix_mem(cond, cond_len, "gitdir:", &cond, &cond_len))
return include_by_gitdir(opts, cond, cond_len, 0); return include_by_gitdir(cf, opts, cond, cond_len, 0);
else if (skip_prefix_mem(cond, cond_len, "gitdir/i:", &cond, &cond_len)) else if (skip_prefix_mem(cond, cond_len, "gitdir/i:", &cond, &cond_len))
return include_by_gitdir(opts, cond, cond_len, 1); return include_by_gitdir(cf, opts, cond, cond_len, 1);
else if (skip_prefix_mem(cond, cond_len, "onbranch:", &cond, &cond_len)) else if (skip_prefix_mem(cond, cond_len, "onbranch:", &cond, &cond_len))
return include_by_branch(cond, cond_len); return include_by_branch(cond, cond_len);
else if (skip_prefix_mem(cond, cond_len, "hasconfig:remote.*.url:", &cond, else if (skip_prefix_mem(cond, cond_len, "hasconfig:remote.*.url:", &cond,
@ -441,16 +446,16 @@ static int git_config_include(const char *var, const char *value, void *data)
return ret; return ret;
if (!strcmp(var, "include.path")) if (!strcmp(var, "include.path"))
ret = handle_path_include(value, inc); ret = handle_path_include(cf_global, value, inc);
if (!parse_config_key(var, "includeif", &cond, &cond_len, &key) && if (!parse_config_key(var, "includeif", &cond, &cond_len, &key) &&
cond && include_condition_is_true(inc, cond, cond_len) && cond && include_condition_is_true(cf_global, inc, cond, cond_len) &&
!strcmp(key, "path")) { !strcmp(key, "path")) {
config_fn_t old_fn = inc->fn; config_fn_t old_fn = inc->fn;
if (inc->opts->unconditional_remote_url) if (inc->opts->unconditional_remote_url)
inc->fn = forbid_remote_url; inc->fn = forbid_remote_url;
ret = handle_path_include(value, inc); ret = handle_path_include(cf_global, value, inc);
inc->fn = old_fn; inc->fn = old_fn;
} }
@ -713,9 +718,9 @@ int git_config_from_parameters(config_fn_t fn, void *data)
struct config_source source; struct config_source source;
memset(&source, 0, sizeof(source)); memset(&source, 0, sizeof(source));
source.prev = cf; source.prev = cf_global;
source.origin_type = CONFIG_ORIGIN_CMDLINE; source.origin_type = CONFIG_ORIGIN_CMDLINE;
cf = &source; cf_global = &source;
env = getenv(CONFIG_COUNT_ENVIRONMENT); env = getenv(CONFIG_COUNT_ENVIRONMENT);
if (env) { if (env) {
@ -773,11 +778,11 @@ out:
strbuf_release(&envvar); strbuf_release(&envvar);
strvec_clear(&to_free); strvec_clear(&to_free);
free(envw); free(envw);
cf = source.prev; cf_global = source.prev;
return ret; return ret;
} }
static int get_next_char(void) static int get_next_char(struct config_source *cf)
{ {
int c = cf->do_fgetc(cf); int c = cf->do_fgetc(cf);
@ -813,13 +818,13 @@ static int get_next_char(void)
return c; return c;
} }
static char *parse_value(void) static char *parse_value(struct config_source *cf)
{ {
int quote = 0, comment = 0, space = 0; int quote = 0, comment = 0, space = 0;
strbuf_reset(&cf->value); strbuf_reset(&cf->value);
for (;;) { for (;;) {
int c = get_next_char(); int c = get_next_char(cf);
if (c == '\n') { if (c == '\n') {
if (quote) { if (quote) {
cf->linenr--; cf->linenr--;
@ -843,7 +848,7 @@ static char *parse_value(void)
for (; space; space--) for (; space; space--)
strbuf_addch(&cf->value, ' '); strbuf_addch(&cf->value, ' ');
if (c == '\\') { if (c == '\\') {
c = get_next_char(); c = get_next_char(cf);
switch (c) { switch (c) {
case '\n': case '\n':
continue; continue;
@ -874,7 +879,8 @@ static char *parse_value(void)
} }
} }
static int get_value(config_fn_t fn, void *data, struct strbuf *name) static int get_value(struct config_source *cf, config_fn_t fn, void *data,
struct strbuf *name)
{ {
int c; int c;
char *value; char *value;
@ -882,7 +888,7 @@ static int get_value(config_fn_t fn, void *data, struct strbuf *name)
/* Get the full name */ /* Get the full name */
for (;;) { for (;;) {
c = get_next_char(); c = get_next_char(cf);
if (cf->eof) if (cf->eof)
break; break;
if (!iskeychar(c)) if (!iskeychar(c))
@ -891,13 +897,13 @@ static int get_value(config_fn_t fn, void *data, struct strbuf *name)
} }
while (c == ' ' || c == '\t') while (c == ' ' || c == '\t')
c = get_next_char(); c = get_next_char(cf);
value = NULL; value = NULL;
if (c != '\n') { if (c != '\n') {
if (c != '=') if (c != '=')
return -1; return -1;
value = parse_value(); value = parse_value(cf);
if (!value) if (!value)
return -1; return -1;
} }
@ -913,13 +919,14 @@ static int get_value(config_fn_t fn, void *data, struct strbuf *name)
return ret; return ret;
} }
static int get_extended_base_var(struct strbuf *name, int c) static int get_extended_base_var(struct config_source *cf, struct strbuf *name,
int c)
{ {
cf->subsection_case_sensitive = 0; cf->subsection_case_sensitive = 0;
do { do {
if (c == '\n') if (c == '\n')
goto error_incomplete_line; goto error_incomplete_line;
c = get_next_char(); c = get_next_char(cf);
} while (isspace(c)); } while (isspace(c));
/* We require the format to be '[base "extension"]' */ /* We require the format to be '[base "extension"]' */
@ -928,13 +935,13 @@ static int get_extended_base_var(struct strbuf *name, int c)
strbuf_addch(name, '.'); strbuf_addch(name, '.');
for (;;) { for (;;) {
int c = get_next_char(); int c = get_next_char(cf);
if (c == '\n') if (c == '\n')
goto error_incomplete_line; goto error_incomplete_line;
if (c == '"') if (c == '"')
break; break;
if (c == '\\') { if (c == '\\') {
c = get_next_char(); c = get_next_char(cf);
if (c == '\n') if (c == '\n')
goto error_incomplete_line; goto error_incomplete_line;
} }
@ -942,7 +949,7 @@ static int get_extended_base_var(struct strbuf *name, int c)
} }
/* Final ']' */ /* Final ']' */
if (get_next_char() != ']') if (get_next_char(cf) != ']')
return -1; return -1;
return 0; return 0;
error_incomplete_line: error_incomplete_line:
@ -950,17 +957,17 @@ error_incomplete_line:
return -1; return -1;
} }
static int get_base_var(struct strbuf *name) static int get_base_var(struct config_source *cf, struct strbuf *name)
{ {
cf->subsection_case_sensitive = 1; cf->subsection_case_sensitive = 1;
for (;;) { for (;;) {
int c = get_next_char(); int c = get_next_char(cf);
if (cf->eof) if (cf->eof)
return -1; return -1;
if (c == ']') if (c == ']')
return 0; return 0;
if (isspace(c)) if (isspace(c))
return get_extended_base_var(name, c); return get_extended_base_var(cf, name, c);
if (!iskeychar(c) && c != '.') if (!iskeychar(c) && c != '.')
return -1; return -1;
strbuf_addch(name, tolower(c)); strbuf_addch(name, tolower(c));
@ -973,7 +980,8 @@ struct parse_event_data {
const struct config_options *opts; const struct config_options *opts;
}; };
static int do_event(enum config_event_t type, struct parse_event_data *data) static int do_event(struct config_source *cf, enum config_event_t type,
struct parse_event_data *data)
{ {
size_t offset; size_t offset;
@ -1004,8 +1012,8 @@ static int do_event(enum config_event_t type, struct parse_event_data *data)
return 0; return 0;
} }
static int git_parse_source(config_fn_t fn, void *data, static int git_parse_source(struct config_source *cf, config_fn_t fn,
const struct config_options *opts) void *data, const struct config_options *opts)
{ {
int comment = 0; int comment = 0;
size_t baselen = 0; size_t baselen = 0;
@ -1024,7 +1032,7 @@ static int git_parse_source(config_fn_t fn, void *data,
for (;;) { for (;;) {
int c; int c;
c = get_next_char(); c = get_next_char(cf);
if (bomptr && *bomptr) { if (bomptr && *bomptr) {
/* We are at the file beginning; skip UTF8-encoded BOM /* We are at the file beginning; skip UTF8-encoded BOM
* if present. Sane editors won't put this in on their * if present. Sane editors won't put this in on their
@ -1042,11 +1050,11 @@ static int git_parse_source(config_fn_t fn, void *data,
} }
if (c == '\n') { if (c == '\n') {
if (cf->eof) { if (cf->eof) {
if (do_event(CONFIG_EVENT_EOF, &event_data) < 0) if (do_event(cf, CONFIG_EVENT_EOF, &event_data) < 0)
return -1; return -1;
return 0; return 0;
} }
if (do_event(CONFIG_EVENT_WHITESPACE, &event_data) < 0) if (do_event(cf, CONFIG_EVENT_WHITESPACE, &event_data) < 0)
return -1; return -1;
comment = 0; comment = 0;
continue; continue;
@ -1054,23 +1062,23 @@ static int git_parse_source(config_fn_t fn, void *data,
if (comment) if (comment)
continue; continue;
if (isspace(c)) { if (isspace(c)) {
if (do_event(CONFIG_EVENT_WHITESPACE, &event_data) < 0) if (do_event(cf, CONFIG_EVENT_WHITESPACE, &event_data) < 0)
return -1; return -1;
continue; continue;
} }
if (c == '#' || c == ';') { if (c == '#' || c == ';') {
if (do_event(CONFIG_EVENT_COMMENT, &event_data) < 0) if (do_event(cf, CONFIG_EVENT_COMMENT, &event_data) < 0)
return -1; return -1;
comment = 1; comment = 1;
continue; continue;
} }
if (c == '[') { if (c == '[') {
if (do_event(CONFIG_EVENT_SECTION, &event_data) < 0) if (do_event(cf, CONFIG_EVENT_SECTION, &event_data) < 0)
return -1; return -1;
/* Reset prior to determining a new stem */ /* Reset prior to determining a new stem */
strbuf_reset(var); strbuf_reset(var);
if (get_base_var(var) < 0 || var->len < 1) if (get_base_var(cf, var) < 0 || var->len < 1)
break; break;
strbuf_addch(var, '.'); strbuf_addch(var, '.');
baselen = var->len; baselen = var->len;
@ -1079,7 +1087,7 @@ static int git_parse_source(config_fn_t fn, void *data,
if (!isalpha(c)) if (!isalpha(c))
break; break;
if (do_event(CONFIG_EVENT_ENTRY, &event_data) < 0) if (do_event(cf, CONFIG_EVENT_ENTRY, &event_data) < 0)
return -1; return -1;
/* /*
@ -1089,11 +1097,11 @@ static int git_parse_source(config_fn_t fn, void *data,
*/ */
strbuf_setlen(var, baselen); strbuf_setlen(var, baselen);
strbuf_addch(var, tolower(c)); strbuf_addch(var, tolower(c));
if (get_value(fn, data, var) < 0) if (get_value(cf, fn, data, var) < 0)
break; break;
} }
if (do_event(CONFIG_EVENT_ERROR, &event_data) < 0) if (do_event(cf, CONFIG_EVENT_ERROR, &event_data) < 0)
return -1; return -1;
switch (cf->origin_type) { switch (cf->origin_type) {
@ -1266,7 +1274,8 @@ int git_parse_ssize_t(const char *value, ssize_t *ret)
} }
NORETURN NORETURN
static void die_bad_number(const char *name, const char *value) static void die_bad_number(struct config_source *cf, const char *name,
const char *value)
{ {
const char *error_type = (errno == ERANGE) ? const char *error_type = (errno == ERANGE) ?
N_("out of range") : N_("invalid unit"); N_("out of range") : N_("invalid unit");
@ -1304,7 +1313,7 @@ int git_config_int(const char *name, const char *value)
{ {
int ret; int ret;
if (!git_parse_int(value, &ret)) if (!git_parse_int(value, &ret))
die_bad_number(name, value); die_bad_number(cf_global, name, value);
return ret; return ret;
} }
@ -1312,7 +1321,7 @@ int64_t git_config_int64(const char *name, const char *value)
{ {
int64_t ret; int64_t ret;
if (!git_parse_int64(value, &ret)) if (!git_parse_int64(value, &ret))
die_bad_number(name, value); die_bad_number(cf_global, name, value);
return ret; return ret;
} }
@ -1320,7 +1329,7 @@ unsigned long git_config_ulong(const char *name, const char *value)
{ {
unsigned long ret; unsigned long ret;
if (!git_parse_ulong(value, &ret)) if (!git_parse_ulong(value, &ret))
die_bad_number(name, value); die_bad_number(cf_global, name, value);
return ret; return ret;
} }
@ -1328,7 +1337,7 @@ ssize_t git_config_ssize_t(const char *name, const char *value)
{ {
ssize_t ret; ssize_t ret;
if (!git_parse_ssize_t(value, &ret)) if (!git_parse_ssize_t(value, &ret))
die_bad_number(name, value); die_bad_number(cf_global, name, value);
return ret; return ret;
} }
@ -1940,20 +1949,20 @@ static int do_config_from(struct config_source *top, config_fn_t fn, void *data,
int ret; int ret;
/* push config-file parsing state stack */ /* push config-file parsing state stack */
top->prev = cf; top->prev = cf_global;
top->linenr = 1; top->linenr = 1;
top->eof = 0; top->eof = 0;
top->total_len = 0; top->total_len = 0;
strbuf_init(&top->value, 1024); strbuf_init(&top->value, 1024);
strbuf_init(&top->var, 1024); strbuf_init(&top->var, 1024);
cf = top; cf_global = top;
ret = git_parse_source(fn, data, opts); ret = git_parse_source(top, fn, data, opts);
/* pop config-file parsing state stack */ /* pop config-file parsing state stack */
strbuf_release(&top->value); strbuf_release(&top->value);
strbuf_release(&top->var); strbuf_release(&top->var);
cf = top->prev; cf_global = top->prev;
return ret; return ret;
} }
@ -2334,12 +2343,12 @@ static int configset_add_value(struct config_set *cs, const char *key, const cha
l_item->e = e; l_item->e = e;
l_item->value_index = e->value_list.nr - 1; l_item->value_index = e->value_list.nr - 1;
if (!cf) if (!cf_global)
BUG("configset_add_value has no source"); BUG("configset_add_value has no source");
if (cf->name) { if (cf_global->name) {
kv_info->filename = strintern(cf->name); kv_info->filename = strintern(cf_global->name);
kv_info->linenr = cf->linenr; kv_info->linenr = cf_global->linenr;
kv_info->origin_type = cf->origin_type; kv_info->origin_type = cf_global->origin_type;
} else { } else {
/* for values read from `git_config_from_parameters()` */ /* for values read from `git_config_from_parameters()` */
kv_info->filename = NULL; kv_info->filename = NULL;
@ -2891,6 +2900,12 @@ static int store_aux_event(enum config_event_t type,
size_t begin, size_t end, void *data) size_t begin, size_t end, void *data)
{ {
struct config_store_data *store = data; struct config_store_data *store = data;
/*
* FIXME Keep using "cf" so that we can avoid rewrapping a
* really long line below. Remove this when "cf" gets plumbed
* correctly.
*/
struct config_source *cf = cf_global;
ALLOC_GROW(store->parsed, store->parsed_nr + 1, store->parsed_alloc); ALLOC_GROW(store->parsed, store->parsed_nr + 1, store->parsed_alloc);
store->parsed[store->parsed_nr].begin = begin; store->parsed[store->parsed_nr].begin = begin;
@ -3771,8 +3786,8 @@ const char *current_config_origin_type(void)
int type; int type;
if (current_config_kvi) if (current_config_kvi)
type = current_config_kvi->origin_type; type = current_config_kvi->origin_type;
else if(cf) else if(cf_global)
type = cf->origin_type; type = cf_global->origin_type;
else else
BUG("current_config_origin_type called outside config callback"); BUG("current_config_origin_type called outside config callback");
@ -3817,8 +3832,8 @@ const char *current_config_name(void)
const char *name; const char *name;
if (current_config_kvi) if (current_config_kvi)
name = current_config_kvi->filename; name = current_config_kvi->filename;
else if (cf) else if (cf_global)
name = cf->name; name = cf_global->name;
else else
BUG("current_config_name called outside config callback"); BUG("current_config_name called outside config callback");
return name ? name : ""; return name ? name : "";
@ -3837,7 +3852,7 @@ int current_config_line(void)
if (current_config_kvi) if (current_config_kvi)
return current_config_kvi->linenr; return current_config_kvi->linenr;
else else
return cf->linenr; return cf_global->linenr;
} }
int lookup_config(const char **mapping, int nr_mapping, const char *var) int lookup_config(const char **mapping, int nr_mapping, const char *var)