From: Christian Couder <christian.couder@gmail.com>
Date: Fri, 01 Aug 2025 10:18:39 GMT
Subject: Re: [PATCH v6 1/4] last-modified: new subcommand to show when files were last modified
Message-ID: <CAP8UFD0AEKDmvQJTanL+ZS+U66WAZz=FKSJ3LPE1QHSEyH-zNw@mail.gmail.com>
In-Reply-To: <20250730175510.987383-2-toon@iotcl.com>

On Wed, Jul 30, 2025 at 7:55 PM Toon Claes <toon@iotcl.com> wrote:

> +[--] <path>...::
> +       For each _<path>_ given, the commit which last modified it is returned.
> +       Without an optional path parameter, all files and subdirectories
> +       in path traversal the are included in the output.

s/the are included/are included/

> +static void last_modified_release(struct last_modified *lm)

I think these days we tend to name those functions using "clear"
instead of "release"

> +{
> +       hashmap_clear_and_free(&lm->paths, struct last_modified_entry, hashent);
> +       release_revisions(&lm->rev);
> +}