From owner-svn-src-head@FreeBSD.ORG Sun Dec 18 01:39:06 2011 Return-Path: Delivered-To: svn-src-head@FreeBSD.ORG Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id DF2B2106564A; Sun, 18 Dec 2011 01:39:06 +0000 (UTC) (envelope-from das@FreeBSD.ORG) Received: from zim.MIT.EDU (ZIM.MIT.EDU [18.95.3.101]) by mx1.freebsd.org (Postfix) with ESMTP id A29348FC14; Sun, 18 Dec 2011 01:39:06 +0000 (UTC) Received: from zim.MIT.EDU (localhost [127.0.0.1]) by zim.MIT.EDU (8.14.5/8.14.2) with ESMTP id pBI1d5Nx021024; Sat, 17 Dec 2011 20:39:05 -0500 (EST) (envelope-from das@FreeBSD.ORG) Received: (from das@localhost) by zim.MIT.EDU (8.14.5/8.14.2/Submit) id pBI1d5YS021023; Sat, 17 Dec 2011 20:39:05 -0500 (EST) (envelope-from das@FreeBSD.ORG) Date: Sat, 17 Dec 2011 20:39:05 -0500 From: David Schultz To: Dimitry Andric Message-ID: <20111218013905.GA20867@zim.MIT.EDU> Mail-Followup-To: Dimitry Andric , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org References: <201112172232.pBHMW1Bd079555@svn.freebsd.org> <4EED18B5.8000907@FreeBSD.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4EED18B5.8000907@FreeBSD.org> Cc: svn-src-head@FreeBSD.ORG, svn-src-all@FreeBSD.ORG, src-committers@FreeBSD.ORG Subject: Re: svn commit: r228668 - head/usr.bin/netstat X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 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: Sun, 18 Dec 2011 01:39:07 -0000 On Sat, Dec 17, 2011, Dimitry Andric wrote: > On 2011-12-17 23:32, Dimitry Andric wrote: > > Author: dim > > Date: Sat Dec 17 22:32:00 2011 > > New Revision: 228668 > > URL: http://svn.freebsd.org/changeset/base/228668 > > > > Log: > > Revert r228650, and work around the clang false positive with printf > > formats in usr.bin/netstat/atalk.c by conditionally adding NO_WFORMAT to > > the Makefile instead. > > > > MFC after: 1 week > > Requested by: bz Have you been keeping track of the other hacks you've been sprinkling throughout the tree to work around clang bugs, e.g., the one in fsdb? It would be unfortunate if someone else has to waste their time later on figuring out what you did, when we could just as easily have waited a month for the clang bug to be fixed. Incidentally, the "bug" you fixed in telnet/utilities.c is also a false positive; clang doesn't understand that an index into a string constant is also a string constant. By the way, I think it's great that you've found so many actual bugs in the tree. I'm just complaining about a small subset of the changes, which fixed non-bugs. :)