From owner-svn-src-head@FreeBSD.ORG Tue Apr 21 19:49:09 2009 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 8FD08106566B; Tue, 21 Apr 2009 19:49:09 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from cyrus.watson.org (cyrus.watson.org [65.122.17.42]) by mx1.freebsd.org (Postfix) with ESMTP id 688C08FC25; Tue, 21 Apr 2009 19:49:09 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from fledge.watson.org (fledge.watson.org [65.122.17.41]) by cyrus.watson.org (Postfix) with ESMTPS id 241C146B06; Tue, 21 Apr 2009 15:49:09 -0400 (EDT) Date: Tue, 21 Apr 2009 20:49:09 +0100 (BST) From: Robert Watson X-X-Sender: robert@fledge.watson.org To: David Schultz In-Reply-To: <20090421194622.GA19215@zim.MIT.EDU> Message-ID: References: <200904201819.n3KIJcZo054306@svn.freebsd.org> <20090421185436.GA18628@zim.MIT.EDU> <20090421190651.GA2505@freebsd.org> <20090421194622.GA19215@zim.MIT.EDU> User-Agent: Alpine 2.00 (BSF 1167 2008-08-23) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: svn-src-head@FreeBSD.ORG, Roman Divacky , src-committers@FreeBSD.ORG, svn-src-all@FreeBSD.ORG Subject: Re: svn commit: r191330 - head/usr.bin/ncal 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: Tue, 21 Apr 2009 19:49:10 -0000 On Tue, 21 Apr 2009, David Schultz wrote: > On Tue, Apr 21, 2009, Roman Divacky wrote: >>> Also, before this change, ncal was already full of convoluted buffer >>> handling, arbitrary buffer sizes, and little to no bounds checking. This >>> commit adds more magic numbers and fragile buffer handling code, and >>> generally makes an already hairy program even less scrutable. This isn't >>> your fault, but it would be nice if we could make ncal better before it >>> gets much worse. For instance, you might use snprintf() or asprintf() >>> instead of an extra half dozen calls to memcpy() with various offsets. >> >> yes, thats true. do you want me to revert this? I am perfectly fine with >> having locally modified cal that supports this highlighting and not share >> this with world at all. > > I don't care (although some other people on this thread seem to); I'm just > encouraging you to clean things up a little before making the code even less > maintainable. The usual moral seems to apply: people who make cosmetic changes should expect cosmetic criticisms. If they aren't happy to receive the criticism, they had best refrain from the changes. Likewise modifying style(9), making gratuitous style changes, re-spelling computer science non-words in comments, etc. Robert N M Watson Computer Laboratory University of Cambridge