git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 0/2] -Wunused-parameter clenaups on tb/multi-pack-bitmaps

From
Taylor Blau <me@ttaylorr.com>
Date
Sep 9, 2021, 20:46 UTC
Message-ID
<YTpyp8ul6ipgDAhY@nand.local>
In-Reply-To
<YTpms9UnVsx+uu/2@coredump.intra.peff.net>
On Thu, Sep 09, 2021 at 03:55:31PM -0400, Jeff King wrote:
> Now that tb/multi-pack-bitmaps is in "next", my local -Wunused-parameter
> topic noticed a few possible cleanups. Neither is a big deal, and I can
> definitely hold them for a larger cleanup. But I thought it might be
> worth doing them while they're fresh in our minds.

Thanks, I'd be happy to see this picked up while the topic is still in next. Both of the spots you touched looked good to me (and don't suddenly become necessary in any of the later series that I'm sitting on).

    Reviewed-by: Taylor Blau <me@ttaylorr.com>

Thanks, Taylor

Previous: Jeff King
Message 4 of 4 in “-Wunused-parameter clenaups on tb/multi-pack-bitmaps”
  1. 0/2 -Wunused-parameter clenaups on tb/multi-pack-bitmapsJeff King, Sep 9, 2021
  2. 1/2 pack-bitmap: drop repository argument from prepare_midx_bitmap_git()Jeff King, Sep 9, 2021
  3. 2/2 pack-bitmap: drop bitmap_index argument from try_partial_reuse()Jeff King, Sep 9, 2021
  4. Taylor BlauSep 9, 2021

Read the whole thread, see it on lore, or plain text.

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