In particular, sparse complains as follows: SP dir.c dir.c:291:12: warning: symbol 'wildcards' was not declared. \ Should it be static? In order to suppress the warning, since 'wildcards' does not need to be an external symbol, we simply add the static modifier to the declaration. Signed-off-by: Ramsay Jones <ramsay@xxxxxxxxxxxxxxxxxxx> --- Hi Nguyen, If you need to re-roll the 'nd/exclude-workaround-top-heavy' branch, could you please squash this into commit dedbd80c ("exclude: do strcmp as much as possible before fnmatch", 26-05-2012). Thanks! ATB, Ramsay Jones dir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dir.c b/dir.c index 1b1a325..571eb0d 100644 --- a/dir.c +++ b/dir.c @@ -288,7 +288,7 @@ int match_pathspec_depth(const struct pathspec *ps, return retval; } -const char *wildcards = "*?[{\\"; +static const char *wildcards = "*?[{\\"; static int no_wildcard(const char *string) { -- 1.7.10 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html