threads / patch / 29216

patchbuiltin/init-db.c: eliminate -Wformat warning on Solaris

Subject: [PATCH] builtin/init-db.c: eliminate -Wformat warning on Solaris

## tl;dr

2 messages between Dec 20, 2011 and Dec 21, 2011. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Ævar Arnfjörð Bjarmason· Dec 20, 2011, 23:27 UTC · lore

On Solaris systems we'd warn about an implicit cast of mode_t when we printed things out with the %d format. We'd get this warning under GCC 4.6.0 with Solaris headers:

    builtin/init-db.c: In function ‘separate_git_dir’:
    builtin/init-db.c:354:4: warning: format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘mode_t’ [-Wformat]

We've been doing this ever since v1.7.4.1-296-gb57fb80. Just work around this by adding an explicit cast.

Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
 builtin/init-db.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Show changes to builtin/init-db.c +1 −1
diff --git a/builtin/init-db.c b/builtin/init-db.c
index d07554c..0dacb8b 100644
--- a/builtin/init-db.c
+++ b/builtin/init-db.c
@@ -351,7 +351,7 @@ static void separate_git_dir(const char *git_dir)
 		else if (S_ISDIR(st.st_mode))
 			src = git_link;
 		else
-			die(_("unable to handle file type %d"), st.st_mode);
+			die(_("unable to handle file type %d"), (int)st.st_mode);
 
 		if (rename(src, git_dir))
 			die_errno(_("unable to move %s to %s"), src, git_dir);
-- 
1.7.7.3
Junio C Hamano· Dec 21, 2011, 00:04 UTC · re: Ævar Arnfjörð Bjarmason · lore

Re: [PATCH] builtin/init-db.c: eliminate -Wformat warning on Solaris

I will queue this directly on 'maint', as I do not think it is worth fixing immediately on top of 2c050e0 (i18n: mark init-db messages for translation, 2011-04-10) and merging all the way down to the now-ancient v1.7.5.X series.

Thanks.

← back to recent threads