Skip to content

Commit 8109984

Browse files
rscharfegitster
authored andcommitted
use strbuf_addbuf() for appending a strbuf to another
Use strbuf_addbuf() where possible; it's shorter and more efficient. Signed-off-by: Rene Scharfe <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent e465796 commit 8109984

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

dir.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2346,7 +2346,7 @@ void write_untracked_extension(struct strbuf *out, struct untracked_cache *untra
23462346

23472347
varint_len = encode_varint(untracked->ident.len, varbuf);
23482348
strbuf_add(out, varbuf, varint_len);
2349-
strbuf_add(out, untracked->ident.buf, untracked->ident.len);
2349+
strbuf_addbuf(out, &untracked->ident);
23502350

23512351
strbuf_add(out, ouc, ouc_size(len));
23522352
free(ouc);

path.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -246,7 +246,7 @@ static void do_submodule_path(struct strbuf *buf, const char *path,
246246
strbuf_addstr(buf, git_dir);
247247
}
248248
strbuf_addch(buf, '/');
249-
strbuf_addstr(&git_submodule_dir, buf->buf);
249+
strbuf_addbuf(&git_submodule_dir, buf);
250250

251251
strbuf_vaddf(buf, fmt, args);
252252

wt-status.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1060,7 +1060,7 @@ static void abbrev_sha1_in_line(struct strbuf *line)
10601060
strbuf_addf(split[1], "%s ", abbrev);
10611061
strbuf_reset(line);
10621062
for (i = 0; split[i]; i++)
1063-
strbuf_addf(line, "%s", split[i]->buf);
1063+
strbuf_addbuf(line, split[i]);
10641064
}
10651065
}
10661066
for (i = 0; split[i]; i++)

0 commit comments

Comments
 (0)