Thread (2 messages) 2 messages, 2 authors, 2024-07-01

Re: [PATCH] submodule--helper: use strvec_pushf() for --super-prefix

From: Jeff King <hidden>
Date: 2024-07-01 03:42:21

On Sat, Jun 29, 2024 at 08:01:24PM +0200, René Scharfe wrote:
Use the strvec_pushf() call that already appends a slash to also produce
the stuck form of the option --super-prefix instead of adding the option
name in a separate call of strvec_push() or strvec_pushl().  This way we
can more easily see that these parts make up a single option with its
argument and save a function call.
I agree it's more readable (both the code and any GIT_TRACE output). But
also I think the "stuck" form is what we recommend in gitcli(7), since
it is future-proof against optional arguments. So we'd also be setting a
good example.

-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