Re: [PATCH v6 1/4] last-modified: new subcommand to show when files were last modified
- From
Christian Couder <christian.couder@gmail.com>
- Date
- Aug 1, 2025, 10:18 UTC
- 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);
> +}