diff options
author | Allan McRae <allan@archlinux.org> | 2013-02-10 16:47:39 +1000 |
---|---|---|
committer | Allan McRae <allan@archlinux.org> | 2013-02-13 11:50:33 +1000 |
commit | 3fa2830829de2e3aa4bb916f164ae2fd75542ef3 (patch) | |
tree | f3bc49fdda7a506314e3fadac7bab37328e31dc8 /src/pacman/util.c | |
parent | 05b0d7e799416b48bb8b40ea199896e2b988698c (diff) | |
download | pacman-3fa2830829de2e3aa4bb916f164ae2fd75542ef3.tar.xz |
Add format attributes to all required functions
Fixes all clang warnings with -Wformat-literal.
Also, fix genuine formating issue discovered once adding these attributes
and add a cast to prevent a gcc warning.
Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'src/pacman/util.c')
-rw-r--r-- | src/pacman/util.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/pacman/util.c b/src/pacman/util.c index 03063fb5..3270c746 100644 --- a/src/pacman/util.c +++ b/src/pacman/util.c @@ -871,7 +871,7 @@ static void _display_targets(alpm_list_t *targets, int verbose) } /* print to screen */ - pm_asprintf(&str, _("Packages (%d):"), alpm_list_count(targets)); + pm_asprintf(&str, _("Packages (%zd):"), alpm_list_count(targets)); printf("\n"); cols = getcols(fileno(stdout)); @@ -992,7 +992,7 @@ static char *pkg_get_location(alpm_pkg_t *pkg) case PM_OP_SYNC: servers = alpm_db_get_servers(alpm_pkg_get_db(pkg)); if(servers) { - pm_asprintf(&string, "%s/%s", servers->data, + pm_asprintf(&string, "%s/%s", (char *)(servers->data), alpm_pkg_get_filename(pkg)); return string; } @@ -1443,6 +1443,7 @@ int select_question(int count) /* presents a prompt and gets a Y/N answer */ +__attribute__((format(printf, 2, 0))) static int question(short preset, char *fmt, va_list args) { char response[32]; |