On Sun, Dec 25 2022, Junio C Hamano wrote: > That would actually make the intent much clearer and if the code > were written to update argc instead of introducing a separate > varilable, this bug would not have happened. Here's a couple of general improvements on top of ab/bundle-wo-args, we should also report to the user that they were missing the <file> argument, as 2/2 here fixes. Ævar Arnfjörð Bjarmason (2): builtin/bundle.c: remove superfluous "newargc" variable bundle <cmd>: have usage_msg_opt() note the missing "<file>" builtin/bundle.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) -- 2.39.0.1106.gf45ba805d1a
