At 2024-05-29 01:24:35, "Junio C Hamano" <gitster@xxxxxxxxx> wrote: [snip] >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
