aboutsummaryrefslogtreecommitdiffstats
path: root/ui-diff.c
diff options
context:
space:
mode:
authorGravatar John Keeping <john@keeping.me.uk>2013-04-06 19:23:52 (JST)
committerGravatar Jason A. Donenfeld <Jason@zx2c4.com>2013-04-08 23:11:29 (JST)
commited5bd30ebe6921dd22948a3f33a314283f043606 (patch)
tree494f1ac8aae7c2147146fefc9aebcdb1bfa671c0 /ui-diff.c
parentd2e20e38141c882e46eaa77f172fc2ae37a19d3b (diff)
downloadcgit-ed5bd30ebe6921dd22948a3f33a314283f043606.zip
cgit-ed5bd30ebe6921dd22948a3f33a314283f043606.tar.gz
Convert cgit_print_error to a variadic function
This removes many uses of "fmt" which uses a fixed size static pool of fixed size buffers. Instead of relying on these, we now pass around argument lists for as long as possible before using a strbuf to render content of an arbitrary size. Signed-off-by: John Keeping <john@keeping.me.uk>
Diffstat (limited to 'ui-diff.c')
-rw-r--r--ui-diff.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/ui-diff.c b/ui-diff.c
index 1115518..8b38209 100644
--- a/ui-diff.c
+++ b/ui-diff.c
@@ -369,12 +369,12 @@ void cgit_print_diff(const char *new_rev, const char *old_rev,
369 get_sha1(new_rev, new_rev_sha1); 369 get_sha1(new_rev, new_rev_sha1);
370 type = sha1_object_info(new_rev_sha1, &size); 370 type = sha1_object_info(new_rev_sha1, &size);
371 if (type == OBJ_BAD) { 371 if (type == OBJ_BAD) {
372 cgit_print_error(fmt("Bad object name: %s", new_rev)); 372 cgit_print_error("Bad object name: %s", new_rev);
373 return; 373 return;
374 } 374 }
375 commit = lookup_commit_reference(new_rev_sha1); 375 commit = lookup_commit_reference(new_rev_sha1);
376 if (!commit || parse_commit(commit)) { 376 if (!commit || parse_commit(commit)) {
377 cgit_print_error(fmt("Bad commit: %s", sha1_to_hex(new_rev_sha1))); 377 cgit_print_error("Bad commit: %s", sha1_to_hex(new_rev_sha1));
378 return; 378 return;
379 } 379 }
380 380
@@ -388,12 +388,12 @@ void cgit_print_diff(const char *new_rev, const char *old_rev,
388 if (!is_null_sha1(old_rev_sha1)) { 388 if (!is_null_sha1(old_rev_sha1)) {
389 type = sha1_object_info(old_rev_sha1, &size); 389 type = sha1_object_info(old_rev_sha1, &size);
390 if (type == OBJ_BAD) { 390 if (type == OBJ_BAD) {
391 cgit_print_error(fmt("Bad object name: %s", sha1_to_hex(old_rev_sha1))); 391 cgit_print_error("Bad object name: %s", sha1_to_hex(old_rev_sha1));
392 return; 392 return;
393 } 393 }
394 commit2 = lookup_commit_reference(old_rev_sha1); 394 commit2 = lookup_commit_reference(old_rev_sha1);
395 if (!commit2 || parse_commit(commit2)) { 395 if (!commit2 || parse_commit(commit2)) {
396 cgit_print_error(fmt("Bad commit: %s", sha1_to_hex(old_rev_sha1))); 396 cgit_print_error("Bad commit: %s", sha1_to_hex(old_rev_sha1));
397 return; 397 return;
398 } 398 }
399 } 399 }