Re: [PATCH v2 5/5] pack-write: pass hash_algo to internal functions

From: Patrick Steinhardt <ps@pks.im>
Date: Jan 17, 2025, 09:46 UTC
Message-ID: <Z4onCYz5zD0L1_Q-@pks.im>
In-Reply-To: <20250117-kn-the-repo-cleanup-v2-5-a7fdc19688f5@gmail.com>

On Fri, Jan 17, 2025 at 10:20:52AM +0100, Karthik Nayak wrote:

> The internal functions `write_rev_trailer()`, `write_rev_trailer()`,
> `write_mtimes_header()` and write_mtimes_trailer()` use the global
> `the_hash_algo` variable to access the repository's hash function. Pass
> the hash from down as we've added made them available in the previous

This doesn't read quite right -- from where do we want to pass it down? Other than that the series looks good to me, thanks!

Patrick

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.