mirror of
https://github.com/git/git.git
synced 2025-12-12 20:36:24 +01:00
Merge branch 'gc/config-partial-submodule-kvi-fix'
Partially revert a sanity check that the rest of the config code was not ready, to avoid triggering it in a corner case. * gc/config-partial-submodule-kvi-fix: config: don't BUG when both kvi and source are set
This commit is contained in:
6
config.c
6
config.c
@@ -109,8 +109,6 @@ static struct config_reader the_reader;
|
||||
static inline void config_reader_push_source(struct config_reader *reader,
|
||||
struct config_source *top)
|
||||
{
|
||||
if (reader->config_kvi)
|
||||
BUG("source should not be set while iterating a config set");
|
||||
top->prev = reader->source;
|
||||
reader->source = top;
|
||||
}
|
||||
@@ -128,16 +126,12 @@ static inline struct config_source *config_reader_pop_source(struct config_reade
|
||||
static inline void config_reader_set_kvi(struct config_reader *reader,
|
||||
struct key_value_info *kvi)
|
||||
{
|
||||
if (kvi && (reader->source || reader->parsing_scope))
|
||||
BUG("kvi should not be set while parsing a config source");
|
||||
reader->config_kvi = kvi;
|
||||
}
|
||||
|
||||
static inline void config_reader_set_scope(struct config_reader *reader,
|
||||
enum config_scope scope)
|
||||
{
|
||||
if (scope && reader->config_kvi)
|
||||
BUG("scope should only be set when iterating through a config source");
|
||||
reader->parsing_scope = scope;
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user