Commit 1785de4e5c102defa56849403885b3d5ead24a12

Carlos Martín Nieto 2018-02-28T11:46:17

config: move the level field into the header We use it in a few places where we might have a full object or a snapshot so move it to where we can actually access it.

diff --git a/src/config_file.c b/src/config_file.c
index 4d13999..2816a89 100644
--- a/src/config_file.c
+++ b/src/config_file.c
@@ -87,13 +87,12 @@ typedef struct {
 	git_mutex values_mutex;
 	refcounted_strmap *values;
 	const git_repository *repo;
+	git_config_level_t level;
 } diskfile_header;
 
 typedef struct {
 	diskfile_header header;
 
-	git_config_level_t level;
-
 	git_array_t(git_config_parser) readers;
 
 	bool locked;
@@ -270,7 +269,7 @@ static int config_open(git_config_backend *cfg, git_config_level_t level, const 
 	int res;
 	diskfile_backend *b = (diskfile_backend *)cfg;
 
-	b->level = level;
+	b->header.level = level;
 	b->header.repo = repo;
 
 	if ((res = refcounted_strmap_alloc(&b->header.values)) < 0)
@@ -343,7 +342,7 @@ static int config_refresh(git_config_backend *cfg)
 	}
 	git_array_clear(b->file.includes);
 
-	if ((error = config_read(values->values, b->header.repo, &b->file, b->level, 0)) < 0)
+	if ((error = config_read(values->values, b->header.repo, &b->file, b->header.level, 0)) < 0)
 		goto out;
 
 	if ((error = git_mutex_lock(&b->header.values_mutex)) < 0) {
@@ -423,7 +422,7 @@ static int config_iterator_new(
 	if ((error = config_snapshot(&snapshot, backend)) < 0)
 		return error;
 
-	if ((error = snapshot->open(snapshot, b->level, b->header.repo)) < 0)
+	if ((error = snapshot->open(snapshot, b->header.level, b->header.repo)) < 0)
 		return error;
 
 	it = git__calloc(1, sizeof(git_config_file_iter));