On Fri, Apr 9, 2021 at 8:50 AM Dave Hansen <dave.hansen@xxxxxxxxx> wrote: > I also considered passing NULL to mean "I don't care about > nr_succeeded". I mostly avoided it to reduce churn. But, looking at it > here, it does seem cleaner. > > Any objections to moving over to Oscar's suggestion? I like this approach (making *nr_succeeded an optional argument). No objection here.