From owner-svn-src-all@FreeBSD.ORG Sun Dec 18 01:41:45 2011 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 092C6106566C; Sun, 18 Dec 2011 01:41:45 +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 A54E28FC15; Sun, 18 Dec 2011 01:41:44 +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 pBI1fiLc021089; Sat, 17 Dec 2011 20:41:44 -0500 (EST) (envelope-from das@freebsd.org) Received: (from das@localhost) by zim.MIT.EDU (8.14.5/8.14.2/Submit) id pBI1firQ021088; Sat, 17 Dec 2011 20:41:44 -0500 (EST) (envelope-from das@freebsd.org) Date: Sat, 17 Dec 2011 20:41:44 -0500 From: David Schultz To: Dimitry Andric , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Message-ID: <20111218014144.GB20867@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> <20111218013905.GA20867@zim.MIT.EDU> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20111218013905.GA20867@zim.MIT.EDU> Cc: Subject: Re: svn commit: r228668 - head/usr.bin/netstat X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 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: Sun, 18 Dec 2011 01:41:45 -0000 On Sat, Dec 17, 2011, David Schultz wrote: > 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. :) Sorry, one more: In less(1), you cast away a bunch of const qualifiers to fix some warnings, but that seems like a step in the wrong direction. The warnings were complaining about genuinely bad code. Disabling the warnings with casts doesn't make less(1) any better; instead, it guarantees that nobody will ever fix the code. Perhaps the larger question is whether it makes sense to fix non-bugs in contributed code at all. What do we get out of it? Maybe if the contrib software is poorly maintained we'll find a bunch of real bugs that won't be addressed upstream. Otherwise, the diffs are only creating headaches for whoever imports the next version.