On 3/8/2022 12:29 PM, Junio C Hamano wrote:
> "Derrick Stolee via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes:
>
>> diff --git a/bundle.h b/bundle.h
>> index 06009fe6b1f..7fef2108f43 100644
>> --- a/bundle.h
>> +++ b/bundle.h
>> @@ -4,12 +4,14 @@
>> #include "strvec.h"
>> #include "cache.h"
>> #include "string-list.h"
>> +#include "list-objects-filter-options.h"
>>
>> struct bundle_header {
>> unsigned version;
>> struct string_list prerequisites;
>> struct string_list references;
>> const struct git_hash_algo *hash_algo;
>> + struct list_objects_filter_options filter;
>> };
>
> This used to be a pointer to the struct, with "NULL means do not
> filter" semantics, with .nr==0 as BUG(). Which was the same
> justification used when an earlier step added a pointer to the
> filter struct to rev_info.
>
> Should the same logic applies there to make it into an embedded
> struct in rev_info as well?
You're absolutely right. Making the change will make the
range-diff look absolutely horrid, but the change isn't
terribly hard for the most part.
Thanks,
-Stolee
