diff options
| author | Mark Lodato | 2010-09-04 11:30:18 -0400 |
|---|---|---|
| committer | Mark Lodato | 2010-09-04 11:35:38 -0400 |
| commit | e4ddc8f72b5a7d8c55a6c2042c7b7f945ba4b1a2 (patch) | |
| tree | 2ade0bb45e15ad5466ca4d3433201477f0c9214e /ui-tree.c | |
| parent | a4c58d92bfee0cf2e35ab1c5e828a6d790108a69 (diff) | |
| download | cgit-e4ddc8f72b5a7d8c55a6c2042c7b7f945ba4b1a2.tar.gz cgit-e4ddc8f72b5a7d8c55a6c2042c7b7f945ba4b1a2.tar.bz2 cgit-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 'ui-tree.c')
| -rw-r--r-- | ui-tree.c | 4 |
1 files changed, 2 insertions, 2 deletions
| @@ -67,7 +67,7 @@ static void print_binary_buffer(char *buf, unsigned long size) | |||
| 67 | html("<table summary='blob content' class='bin-blob'>\n"); | 67 | html("<table summary='blob content' class='bin-blob'>\n"); |
| 68 | html("<tr><th>ofs</th><th>hex dump</th><th>ascii</th></tr>"); | 68 | html("<tr><th>ofs</th><th>hex dump</th><th>ascii</th></tr>"); |
| 69 | for (ofs = 0; ofs < size; ofs += ROWLEN, buf += ROWLEN) { | 69 | for (ofs = 0; ofs < size; ofs += ROWLEN, buf += ROWLEN) { |
| 70 | htmlf("<tr><td class='right'>%04x</td><td class='hex'>", ofs); | 70 | htmlf("<tr><td class='right'>%04lx</td><td class='hex'>", ofs); |
| 71 | for (idx = 0; idx < ROWLEN && ofs + idx < size; idx++) | 71 | for (idx = 0; idx < ROWLEN && ofs + idx < size; idx++) |
| 72 | htmlf("%*s%02x", | 72 | htmlf("%*s%02x", |
| 73 | idx == 16 ? 4 : 1, "", | 73 | idx == 16 ? 4 : 1, "", |
| @@ -108,7 +108,7 @@ static void print_object(const unsigned char *sha1, char *path, const char *base | |||
| 108 | html(")\n"); | 108 | html(")\n"); |
| 109 | 109 | ||
| 110 | if (ctx.cfg.max_blob_size && size / 1024 > ctx.cfg.max_blob_size) { | 110 | if (ctx.cfg.max_blob_size && size / 1024 > ctx.cfg.max_blob_size) { |
| 111 | htmlf("<div class='error'>blob size (%dKB) exceeds display size limit (%dKB).</div>", | 111 | htmlf("<div class='error'>blob size (%ldKB) exceeds display size limit (%dKB).</div>", |
| 112 | size / 1024, ctx.cfg.max_blob_size); | 112 | size / 1024, ctx.cfg.max_blob_size); |
| 113 | return; | 113 | return; |
| 114 | } | 114 | } |
