Re: [PATCH 07/11] bundle: safely handle --objects option
- From
Derrick Stolee <derrickstolee@github.com>
- Date
- Mar 7, 2022, 14:09 UTC
- Message-ID
- <934e76db-e85b-7d62-e98c-0d1a8450bdaf@github.com>
- In-Reply-To
- <xmqqilstxr3o.fsf@gitster.g>
On 3/4/2022 5:58 PM, Junio C Hamano wrote:
Show 19 quoted lines
> Jeff Hostetler <git@jeffhostetler.com> writes: > >> On 2/23/22 12:55 PM, Derrick Stolee via GitGitGadget wrote: >>> From: Derrick Stolee <derrickstolee@github.com> >>> Since 'git bundle' uses setup_revisions() to specify the object >>> walk, >>> some options do not make sense to include during the pack-objects child >>> process. Further, these options are used for a call to >>> traverse_commit_list() which would then require a callback which is >>> currently NULL. >>> By populating the callback we prevent a segfault in the case of >>> adding >>> the --objects flag. This is really a redundant statement because the >> >> Nit: I stumbled over "...because the bundles are constructing..." >> Is there a better wording here?? > > "... because the command is constructing ..." should be sufficient, > I hope?
That works for me. Thanks! -Stolee