DORMANTno replies

[PATCH] grep.c: Fix a sparse warning

From: Ramsay Jones <hidden>
Date: 2016-06-15 22:54:48
Subsystem: the rest · Maintainer: Linus Torvalds

In particular, sparse complains that "... 'dump_grep_expression'
was not declared. Should it be static?". In order to suppress
the warning, since this function does not need more than file
scope, we simply include the static modifier in it's declaration.

Signed-off-by: Ramsay Jones <redacted>
---

Hi Junio,

I prefer to catch these before they progress to next, but it seems
I've not been quick enough lately! Sorry about that. :(  [I will have
to git-fetch more frequently; at present I only fetch about 3 times
a week.]

I've been away for a few days, so I'm well behind ... (I'm just
about to download 350+ emails for me to read tonight!).

ATB,
Ramsay Jones

 grep.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/grep.c b/grep.c
index 925aa92..38c4d75 100644
--- a/grep.c
+++ b/grep.c
@@ -403,7 +403,7 @@ static void dump_grep_expression_1(struct grep_expr *x, int in)
 	}
 }
 
-void dump_grep_expression(struct grep_opt *opt)
+static void dump_grep_expression(struct grep_opt *opt)
 {
 	struct grep_expr *x = opt->pattern_expression;
 
-- 
1.7.12
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help