From owner-svn-src-head@FreeBSD.ORG Sat Jul 26 18:33:49 2014 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 17190645; Sat, 26 Jul 2014 18:33:49 +0000 (UTC) Received: from mail110.syd.optusnet.com.au (mail110.syd.optusnet.com.au [211.29.132.97]) by mx1.freebsd.org (Postfix) with ESMTP id C9DAB2496; Sat, 26 Jul 2014 18:33:48 +0000 (UTC) Received: from c122-106-147-133.carlnfd1.nsw.optusnet.com.au (c122-106-147-133.carlnfd1.nsw.optusnet.com.au [122.106.147.133]) by mail110.syd.optusnet.com.au (Postfix) with ESMTPS id 7C68C783363; Sun, 27 Jul 2014 04:33:40 +1000 (EST) Date: Sun, 27 Jul 2014 04:33:37 +1000 (EST) From: Bruce Evans X-X-Sender: bde@besplex.bde.org To: Benjamin Kaduk Subject: Re: svn commit: r269112 - head/sys/boot/arm/at91/boot2 In-Reply-To: Message-ID: <20140727040529.X5986@besplex.bde.org> References: <201407260409.s6Q49i20065578@svn.freebsd.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed X-Optus-CM-Score: 0 X-Optus-CM-Analysis: v=2.1 cv=B4eAjodM c=1 sm=1 tr=0 a=7NqvjVvQucbO2RlWB8PEog==:117 a=PO7r1zJSAAAA:8 a=Ur8WnqaOPh0A:10 a=kj9zAlcOel0A:10 a=JzwRw_2MAAAA:8 a=6I5d2MoRAAAA:8 a=D2rKaxmZ94vHiccmnLcA:9 a=CjuIK1q_8ugA:10 a=SV7veod9ZcQA:10 Cc: "svn-src-head@freebsd.org" , "svn-src-all@freebsd.org" , "src-committers@freebsd.org" , Ian Lepore X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 26 Jul 2014 18:33:49 -0000 On Sat, 26 Jul 2014, Benjamin Kaduk wrote: > On Sat, Jul 26, 2014 at 12:09 AM, Ian Lepore wrote: > >> Author: ian >> Date: Sat Jul 26 04:09:43 2014 >> New Revision: 269112 >> URL: http://svnweb.freebsd.org/changeset/base/269112 >> >> Log: >> The 'flags' variable is conflicting with some inline code in a header >> file >> ("variable flags shadows a global..."), just rename this variable to >> wish away the problem. > > Not trying to pick on you, but we have lots of places where we get > "variable shadows a global..." -- we have a global variable 'buf' in > the kernel namespace, for one :) > Do we think it's a good use of our time to try and get this to zero > instances? Not very good, and it is negatively good to unimprove local names because of namespace pollution in global names. The shadowing for 'buf' can only be seen by C compilers that look only at source files for the file being compiled if is included, but namespace pollution makes it more visible. The pollution is relatively small. In freefall's kernel now, according to word counts of .depend, there are approx. 187338 included files only 174 of them are sys/buf.h. The number of nested includes (adjusted for kernel feature expansion) has increased by about a factor of 3 in the last 15 years since I stopped trying to police it. Bruce