Should I still change the commit message regarding where I mention the commit that introduced this, as explained by Eric? From the activity I saw overnight it looks like things were already picked up and it doesn't matter anymore. On Tue, Nov 22, 2022 at 10:21 PM Junio C Hamano <gitster@xxxxxxxxx> wrote: > > Eric Sunshine <sunshine@xxxxxxxxxxxxxx> writes: > > > Makes sense. Well explained. > > > > A separate problem is that chainlint.pl doesn't fall back to a > > sensible non-zero value if ncores() returns 0 (or some other nonsense > > value). That is, of course, outside the scope of the well-focused > > problem fix which this standalone patch addresses. I may end up > > submitting a fix separately to make it fall back sensibly. > > ... > > As mentioned elsewhere[1], this code may eventually be dropped > > altogether, but this fix is good to have in the meantime. Thanks. > > Thanks, both. Will apply.
