commit: use strbuf_add() to add a length-limited string
diff mbox series

Message ID f4c7d65f-d7bb-c574-fe48-8d2df0c2907a@web.de
State New
Headers show
Series
  • commit: use strbuf_add() to add a length-limited string
Related show

Commit Message

René Scharfe Dec. 7, 2019, 11:16 a.m. UTC
This is shorter and simpler.

Signed-off-by: René Scharfe <l.s.r@web.de>
---
 builtin/commit.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.24.0

Comments

Derrick Stolee Dec. 7, 2019, 8:17 p.m. UTC | #1
On 12/7/2019 6:16 AM, René Scharfe wrote:
> This is shorter and simpler.
> 
> Signed-off-by: René Scharfe <l.s.r@web.de>
> ---
>  builtin/commit.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/builtin/commit.c b/builtin/commit.c
> index 294dc574cd..35a1fb9ad6 100644
> --- a/builtin/commit.c
> +++ b/builtin/commit.c
> @@ -537,7 +537,7 @@ static void export_one(const char *var, const char *s, const char *e, int hack)
>  	struct strbuf buf = STRBUF_INIT;
>  	if (hack)
>  		strbuf_addch(&buf, hack);
> -	strbuf_addf(&buf, "%.*s", (int)(e - s), s);
> +	strbuf_add(&buf, s, e - s);

I had to look up what "%.*s" does [1, 2] but this looks like
a behavior-equivalent (and possibly faster) way to do it.

-Stolee

[1] https://stackoverflow.com/a/49580164
[2] https://en.cppreference.com/w/c/io/fprintf
Junio C Hamano Dec. 9, 2019, 7:24 p.m. UTC | #2
René Scharfe <l.s.r@web.de> writes:

> This is shorter and simpler.
>
> Signed-off-by: René Scharfe <l.s.r@web.de>
> ---

Ack (this was from me abck in 2012).

>  builtin/commit.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/builtin/commit.c b/builtin/commit.c
> index 294dc574cd..35a1fb9ad6 100644
> --- a/builtin/commit.c
> +++ b/builtin/commit.c
> @@ -537,7 +537,7 @@ static void export_one(const char *var, const char *s, const char *e, int hack)
>  	struct strbuf buf = STRBUF_INIT;
>  	if (hack)
>  		strbuf_addch(&buf, hack);
> -	strbuf_addf(&buf, "%.*s", (int)(e - s), s);
> +	strbuf_add(&buf, s, e - s);
>  	setenv(var, buf.buf, 1);
>  	strbuf_release(&buf);
>  }
> --
> 2.24.0

Patch
diff mbox series

diff --git a/builtin/commit.c b/builtin/commit.c
index 294dc574cd..35a1fb9ad6 100644
--- a/builtin/commit.c
+++ b/builtin/commit.c
@@ -537,7 +537,7 @@  static void export_one(const char *var, const char *s, const char *e, int hack)
 	struct strbuf buf = STRBUF_INIT;
 	if (hack)
 		strbuf_addch(&buf, hack);
-	strbuf_addf(&buf, "%.*s", (int)(e - s), s);
+	strbuf_add(&buf, s, e - s);
 	setenv(var, buf.buf, 1);
 	strbuf_release(&buf);
 }