From owner-svn-src-all@freebsd.org Fri May 26 01:15:00 2017 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 1752BD82BE0; Fri, 26 May 2017 01:15:00 +0000 (UTC) (envelope-from pfg@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id BEF7E16B7; Fri, 26 May 2017 01:14:59 +0000 (UTC) (envelope-from pfg@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id v4Q1Ewb6025185; Fri, 26 May 2017 01:14:58 GMT (envelope-from pfg@FreeBSD.org) Received: (from pfg@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id v4Q1Ewcp025182; Fri, 26 May 2017 01:14:58 GMT (envelope-from pfg@FreeBSD.org) Message-Id: <201705260114.v4Q1Ewcp025182@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: pfg set sender to pfg@FreeBSD.org using -f From: "Pedro F. Giffuni" Date: Fri, 26 May 2017 01:14:58 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r318913 - head/lib/libc/gen X-SVN-Group: head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 26 May 2017 01:15:00 -0000 Author: pfg Date: Fri May 26 01:14:58 2017 New Revision: 318913 URL: https://svnweb.freebsd.org/changeset/base/318913 Log: fts_open: move bogus initialization further below, before it is used. Move an unneeded initialization, introduced in r54770 to quiet down GCC, to a place nearer to its first use. This has no practical effect, it just keeps the garbage better sorted. Hinted by: OpenBSD (CVS rev. 1.56, without obfuscations) Modified: head/lib/libc/gen/fts-compat.c head/lib/libc/gen/fts-compat11.c head/lib/libc/gen/fts.c Modified: head/lib/libc/gen/fts-compat.c ============================================================================== --- head/lib/libc/gen/fts-compat.c Fri May 26 00:51:05 2017 (r318912) +++ head/lib/libc/gen/fts-compat.c Fri May 26 01:14:58 2017 (r318913) @@ -146,9 +146,6 @@ __fts_open_44bsd(char * const *argv, int sp->fts_compar = compar; sp->fts_options = options; - /* Shush, GCC. */ - tmp = NULL; - /* Logical walks turn on NOCHDIR; symbolic links are too hard. */ if (ISSET(FTS_LOGICAL)) SET(FTS_NOCHDIR); @@ -165,6 +162,9 @@ __fts_open_44bsd(char * const *argv, int goto mem2; parent->fts_level = FTS_ROOTPARENTLEVEL; + /* Shush, GCC. */ + tmp = NULL; + /* Allocate/initialize root(s). */ for (root = NULL, nitems = 0; *argv != NULL; ++argv, ++nitems) { /* Don't allow zero-length paths. */ Modified: head/lib/libc/gen/fts-compat11.c ============================================================================== --- head/lib/libc/gen/fts-compat11.c Fri May 26 00:51:05 2017 (r318912) +++ head/lib/libc/gen/fts-compat11.c Fri May 26 01:14:58 2017 (r318913) @@ -142,9 +142,6 @@ freebsd11_fts_open(char * const *argv, i sp->fts_compar = compar; sp->fts_options = options; - /* Shush, GCC. */ - tmp = NULL; - /* Logical walks turn on NOCHDIR; symbolic links are too hard. */ if (ISSET(FTS_LOGICAL)) SET(FTS_NOCHDIR); @@ -161,6 +158,9 @@ freebsd11_fts_open(char * const *argv, i goto mem2; parent->fts_level = FTS_ROOTPARENTLEVEL; + /* Shush, GCC. */ + tmp = NULL; + /* Allocate/initialize root(s). */ for (root = NULL, nitems = 0; *argv != NULL; ++argv, ++nitems) { len = strlen(*argv); Modified: head/lib/libc/gen/fts.c ============================================================================== --- head/lib/libc/gen/fts.c Fri May 26 00:51:05 2017 (r318912) +++ head/lib/libc/gen/fts.c Fri May 26 01:14:58 2017 (r318913) @@ -137,9 +137,6 @@ fts_open(char * const *argv, int options sp->fts_compar = compar; sp->fts_options = options; - /* Shush, GCC. */ - tmp = NULL; - /* Logical walks turn on NOCHDIR; symbolic links are too hard. */ if (ISSET(FTS_LOGICAL)) SET(FTS_NOCHDIR); @@ -156,6 +153,9 @@ fts_open(char * const *argv, int options goto mem2; parent->fts_level = FTS_ROOTPARENTLEVEL; + /* Shush, GCC. */ + tmp = NULL; + /* Allocate/initialize root(s). */ for (root = NULL, nitems = 0; *argv != NULL; ++argv, ++nitems) { len = strlen(*argv);