diff options
author | John Keeping | 2013-04-06 11:23:52 +0100 |
---|---|---|
committer | Jason A. Donenfeld | 2013-04-08 16:11:29 +0200 |
commit | ed5bd30ebe6921dd22948a3f33a314283f043606 (patch) | |
tree | 494f1ac8aae7c2147146fefc9aebcdb1bfa671c0 /ui-diff.c | |
parent | d2e20e38141c882e46eaa77f172fc2ae37a19d3b (diff) | |
download | cgit-ed5bd30ebe6921dd22948a3f33a314283f043606.tar.gz cgit-ed5bd30ebe6921dd22948a3f33a314283f043606.tar.bz2 cgit-ed5bd30ebe6921dd22948a3f33a314283f043606.zip |
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.c | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -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 | } |