Patrick Steinhardt bdbebe5714 refs: introduce wrapper struct for `each_ref_fn` ... The `each_ref_fn` callback function type is used across our code base for several different functions that iterate through reference. There's a bunch of callbacks implementing this type, which makes any changes to the callback signature extremely noisy. An example of the required churn is e8207717f1 (refs: add referent to each_ref_fn, 2024-08-09): adding a single argument required us to change 48 files. It was already proposed back then [1] that we might want to introduce a wrapper structure to alleviate the pain going forward. While this of course requires the same kind of global refactoring as just introducing a new parameter, it at least allows us to more change the callback type afterwards by just extending the wrapper structure. One counterargument to this refactoring is that it makes the structure more opaque. While it is obvious which callsites need to be fixed up when we change the function type, it's not obvious anymore once we use a structure. That being said, we only have a handful of sites that actually need to populate this wrapper structure: our ref backends, "refs/iterator.c" as well as very few sites that invoke the iterator callback functions directly. Introduce this wrapper structure so that we can adapt the iterator interfaces more readily. [1]: <ZmarVcF5JjsZx0dl@tanuki> Signed-off-by: Patrick Steinhardt <ps@pks.im> Signed-off-by: Junio C Hamano <gitster@pobox.com>		2025-11-04 07:32:24 -08:00
..
default.c	refs: introduce wrapper struct for `each_ref_fn`	2025-11-04 07:32:24 -08:00
default.h	…
noop.c	treewide: remove unnecessary includes in source files	2023-12-26 12:04:31 -08:00
noop.h	negotiator/noop: add noop fetch negotiator	2020-08-18 13:25:05 -07:00
skipping.c	refs: introduce wrapper struct for `each_ref_fn`	2025-11-04 07:32:24 -08:00
skipping.h	…