From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Mon, 04 Jan 2021 15:17:16 GMT
Subject: Re: [PATCH v2 2/5] mailmap: switch to opaque struct
Message-ID: <87ft3gviir.fsf@evledraar.gmail.com>
In-Reply-To: <20210103211849.2691287-3-sandals@crustytoothpaste.net>


On Sun, Jan 03 2021, brian m. carlson wrote:

> -void clear_mailmap(struct string_list *map)
> +void clear_mailmap(struct mailmap *mailmap)
>  {
> +	struct string_list *map = mailmap->mailmap;
>  	debug_mm("mailmap: clearing %d entries...\n", map->nr);

Re my comment on 1/5 wouldn't it make more sense to use the new API
here:

    -       debug_mm("mailmap: clearing %d entries...\n", map->nr);
    +       debug_mm("mailmap: clearing %u entries...\n", mailmap_entries(mailmap));

I changed %d to %u here, maybe we'd also want to do that earlier while
we're at it (not code you're changing):

    -       debug_mm("mailmap: removing entries for <%s>, with %d sub-entries\n",
    +       debug_mm("mailmap: removing entries for <%s>, with %u sub-entries\n",