Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes: > Ævar Arnfjörð Bjarmason (5): > config.h: move up "struct key_value_info" > config.c: use "enum config_origin_type", not "int" > config API: add a config_origin_type_name() helper > config.c: refactor configset_iter() > config API: add and use a repo_config_kvi() I haven't reached the end of 5/5 but it is a shame to see that these obviously good and trivial clean-up patches like [1-4/5] have to be buried in an RFC patch and benefit of applying them alone becomes unclear only because it is done in an area that is actively worked on by others. The latter is unfortunately inherent to the approach of "commenting by patches" but hopefully we can see them reappear when the tree is quiescent. Thanks.