Re:Re: [PATCH v3 3/4] fetch-pack: expose fsckObjects configuration logic
- From
- Xing Xin <bupt_xingxin@163.com>
- Date
- May 30, 2024, 08:48 UTC
- Message-ID
- <5cb91961.8ebd.18fc8b01748.Coremail.bupt_xingxin@163.com>
- In-Reply-To
- <xmqqbk4p6e98.fsf@gitster.g>
At 2024-05-29 01:24:35, "Junio C Hamano" <gitster@pobox.com> wrote: [snip]
Show 16 quoted lines
>The original before it was made into a helper function was written >as a cascade of ?: operators, because it had to be a single >expression. As the body of a helper function, we now can sprinkle >multiple return statements in it. I think the way that is easiest >to understand is > > /* the most specific, if specified */ > if (fetch_fsck_objects >= 0) > return fetch_fsck_objects; > /* the less specific, catch-all for both directions */ > if (transfer_fsck_objects >= 0) > return transfer_fsck_objects; > /* the fallback hardcoded default */ > return 0; > >without the /* comments */.
Applied, thanks!
Xing Xin