Code

builtin/tag.c: Fix a sparse warning
authorRamsay Jones <ramsay@ramsay1.demon.co.uk>
Mon, 13 Feb 2012 18:24:41 +0000 (18:24 +0000)
committerJunio C Hamano <gitster@pobox.com>
Mon, 13 Feb 2012 19:12:36 +0000 (11:12 -0800)
In particular, sparse complains as follows:

        SP builtin/tag.c
    builtin/tag.c:411:5: warning: symbol 'parse_opt_points_at' was \
        not declared. Should it be static?

In order to suppress the warning, since the parse_opt_points_at()
function does not need to be an external symbol, we simply add the
static modifier to the function definition.

Signed-off-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/tag.c

index 27c35571abbd57b95153361b97ae1c1b951793ca..e377706e6d362792055b5e2a67efc12edd972e52 100644 (file)
@@ -400,7 +400,7 @@ static int strbuf_check_tag_ref(struct strbuf *sb, const char *name)
        return check_refname_format(sb->buf, 0);
 }
 
-int parse_opt_points_at(const struct option *opt __attribute__ ((unused)),
+static int parse_opt_points_at(const struct option *opt __attribute__((unused)),
                        const char *arg, int unset)
 {
        unsigned char sha1[20];