From: Ævar Arnfjörð Bjarmason Date: Tue, 27 Dec 2022 18:39:08 GMT Subject: [PATCH 0/2] builtin/bundle.c: segfault fix style & error reporting follow-up Message-ID: In-Reply-To: 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 argument, as 2/2 here fixes. Ævar Arnfjörð Bjarmason (2): builtin/bundle.c: remove superfluous "newargc" variable bundle : have usage_msg_opt() note the missing "" builtin/bundle.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) -- 2.39.0.1106.gf45ba805d1a