svn commit: r334808 - head/usr.bin/grep

Kyle Evans kevans at FreeBSD.org
Thu Jun 7 18:38:49 UTC 2018


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;


More information about the svn-src-head mailing list