Author: kevans
Date: Thu Jun 7 18:38:48 2018
New Revision: 334808
URL: https://svnweb.freebsd.org/changeset/base/334808
Log:
bsdgrep(1): Don't initialize fts_flags twice
Admittedly, this is a clang-scan complaint... but it wasn't wrong. fts_flags
is initialized by all cases in the switch(), which should be fairly obvious.
Annotate this anyways.
Modified:
head/usr.bin/grep/util.c
Modified: head/usr.bin/grep/util.c
==============================================================================
--- head/usr.bin/grep/util.c Thu Jun 7 18:36:12 2018 (r334807)
+++ head/usr.bin/grep/util.c Thu Jun 7 18:38:48 2018 (r334808)
@@ -148,9 +148,9 @@ grep_tree(char **argv)
bool matched, ok;
const char *wd[] = { ".", NULL };
- fts_flags = 0;
matched = false;
+ /* This switch effectively initializes 'fts_flags' */
switch(linkbehave) {
case LINK_EXPLICIT:
fts_flags = FTS_COMFOLLOW;
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
