diff options
author | Mark Lodato <lodatom@gmail.com> | 2010-09-04 11:30:18 -0400 |
---|---|---|
committer | Mark Lodato <lodatom@gmail.com> | 2010-09-04 11:35:38 -0400 |
commit | e4ddc8f72b5a7d8c55a6c2042c7b7f945ba4b1a2 (patch) | |
tree | 2ade0bb45e15ad5466ca4d3433201477f0c9214e /ui-log.c | |
parent | use __attribute__ to catch printf format mistakes (diff) | |
download | cgit-pink-e4ddc8f72b5a7d8c55a6c2042c7b7f945ba4b1a2.tar.gz cgit-pink-e4ddc8f72b5a7d8c55a6c2042c7b7f945ba4b1a2.zip |
fix errors in printf-style format strings
There were many places where the arguments to a printf-like function did not match the format string. Mostly, these were a missing 'l' flag, but there were three exceptions: - In ui-stats.c, a size_t argument must be printed. C99 has the "%zu" flag for this purpose, but not all compilers support this. Therefore, we mimic what git does - use a NO_C99_FORMAT Makefile variable. - In ui-stats.c, cgit_print_error() was called with a pointer instead of a character. - In ui-log.c, the "columns" argument was never used. Signed-off-by: Mark Lodato <lodatom@gmail.com>
Diffstat (limited to '')
-rw-r--r-- | ui-log.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/ui-log.c b/ui-log.c index 0536b23..41b5225 100644 --- a/ui-log.c +++ b/ui-log.c @@ -228,8 +228,7 @@ void cgit_print_log(const char *tip, int ofs, int cnt, char *grep, char *pattern commit->parents = NULL; } if (pager) { - htmlf("</table><div class='pager'>", - columns); + html("</table><div class='pager'>"); if (ofs > 0) { cgit_log_link("[prev]", NULL, NULL, ctx.qry.head, ctx.qry.sha1, ctx.qry.vpath, |