On 10/6/2020 4:38 PM, Derrick Stolee wrote:
> brian m. carlson d7e6b6a8 fast-import: convert internal structs to struct object_id
> brian m. carlson 912c13d5 fast-import: convert to struct object_id
> brian m. carlson e6a492b7 pack: convert struct pack_idx_entry to struct object_id
> brian m. carlson 28d055bd fast-import: make hash-size independent
These are clear refactors that should not be considered against the
author.
> Derrick Stolee 4ddc79b2 maintenance: add auto condition for commit-graph task
Here, it seems I missed testing "git maintenance run --auto --task=commit-graph".
I will send a patch to correct this sometime this week.
> Han-Wen Nienhuys 4441f427 refs: add GIT_TRACE_REFS debugging mechanism
> refs/debug.c
This file seems rather non-trivial to not be covered. However, if GIT_TRACE_REFS
is only intended for debugging, then perhaps none of this is in a critical code
path that would affect normal users?
> Jeff King c33ddc2e date: use strbufs in date-formatting functions> Jeff King d70a9eb6 strvec: rename struct fields
> Jeff King ef8d7ac4 strvec: convert more callers away from argv_array name
> Jeff King c972bf4c strvec: convert remaining callers away from argv_array name
This is another giant refactor.
> Jonathan Tan f08cbf60 index-pack: make quantum of work smaller
> builtin/index-pack.c
> f08cbf60 424) list_for_each_prev(pos, &done_head) {
> f08cbf60 425) struct base_data *b = list_entry(pos, struct base_data, list);
> f08cbf60 426) if (b->retain_data || b == retain)
> f08cbf60 427) continue;
> f08cbf60 428) if (b->data) {
> f08cbf60 429) free_base_data(b);
> f08cbf60 430) if (base_cache_used <= base_cache_limit)
> f08cbf60 431) return;
> f08cbf60 435) list_for_each_prev(pos, &work_head) {
> f08cbf60 436) struct base_data *b = list_entry(pos, struct base_data, list);
> f08cbf60 437) if (b->retain_data || b == retain)
> f08cbf60 438) continue;
> f08cbf60 439) if (b->data) {
> f08cbf60 440) free_base_data(b);
> f08cbf60 441) if (base_cache_used <= base_cache_limit)
> f08cbf60 442) return;
> f08cbf60 925) base_cache_used += c->size;
> f08cbf60 941) base_cache_used += c->size;
This seems to be sufficiently complicated to be worth a test.
What do you think, Jonathan?
These are the items that caught my eye.
Thanks,
-Stolee
