Thread (4 messages) 4 messages, 1 author, 2022-08-25
STALE1376d

[PATCH 0/3] unused function parameters newly in next

From: Jeff King <hidden>
Date: 2022-08-25 10:46:07

The sg/parse-options-subcommand topic was merged to 'next', and it
introduced a ton of -Wunused-parameter warnings. But I think in this
case it revealed some actual bugs, albeit minor.

Here are some fixes. I prepared them on top of that topic, which is
semantically necessary. But note that the topic is not itself at fault
for _introducing_ any bugs. It just revealed them. So I don't think
there's any need to pause its progress to master.

  [1/3]: pass subcommand "prefix" arguments to parse_options()
  [2/3]: maintenance: add parse-options boilerplate for subcommands
  [3/3]: remote: run "remote rm" argv through parse_options()

 builtin/commit-graph.c     |  4 ++--
 builtin/gc.c               | 43 +++++++++++++++++++++++++++++++++++++-
 builtin/multi-pack-index.c |  8 +++----
 builtin/remote.c           | 36 ++++++++++++++++++-------------
 builtin/sparse-checkout.c  |  8 +++----
 5 files changed, 73 insertions(+), 26 deletions(-)

-Peff
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help