diff options
| author | 2013-04-06 19:23:52 (JST) | |
|---|---|---|
| committer | 2013-04-08 23:11:29 (JST) | |
| commit | ed5bd30ebe6921dd22948a3f33a314283f043606 (patch) | |
| tree | 494f1ac8aae7c2147146fefc9aebcdb1bfa671c0 /ui-diff.c | |
| parent | d2e20e38141c882e46eaa77f172fc2ae37a19d3b (diff) | |
| download | cgit-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.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 | } |
