Re: [PATCH 07/20] show_objects_for_type(): convert to new revindex API

From: Taylor Blau <me@ttaylorr.com>
Date: Jan 12, 2021, 16:35 UTC
Message-ID: <X/3P1eM+VDyNWRSY@nand.local>
In-Reply-To: <X/1klwe44go+A+Xi@coredump.intra.peff.net>

On Tue, Jan 12, 2021 at 03:57:59AM -0500, Jeff King wrote:

> A minor nit, but "n" isn't very descriptive. It's not in scope for very
> long, so that's not too bad, but there are two positions at work in this
> function: the pos/offset bit position, and the index position. Maybe
> "index_pos" would be better than "n" to keep the two clear?

Much clearer, thank you.

> -Peff

Thanks, Taylor

$ 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.