Return-Path: Delivered-To: apmail-forrest-dev-archive@www.apache.org Received: (qmail 2374 invoked from network); 22 Dec 2005 01:27:49 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 22 Dec 2005 01:27:49 -0000 Received: (qmail 55506 invoked by uid 500); 22 Dec 2005 01:27:48 -0000 Delivered-To: apmail-forrest-dev-archive@forrest.apache.org Received: (qmail 55479 invoked by uid 500); 22 Dec 2005 01:27:48 -0000 Mailing-List: contact dev-help@forrest.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: List-Post: Reply-To: dev@forrest.apache.org List-Id: Delivered-To: mailing list dev@forrest.apache.org Received: (qmail 55468 invoked by uid 99); 22 Dec 2005 01:27:48 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 21 Dec 2005 17:27:48 -0800 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received-SPF: neutral (asf.osuosl.org: local policy) Received: from [65.77.211.84] (HELO www2.kc.aoindustries.com) (65.77.211.84) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 21 Dec 2005 17:27:47 -0800 Received: from fo2.kc.aoindustries.com (www2.kc.aoindustries.com [65.77.211.84]) by www2.kc.aoindustries.com (8.13.1/8.13.1) with ESMTP id jBM1RQ51022784 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO) for ; Wed, 21 Dec 2005 19:27:26 -0600 Received: from localhost (localhost [[UNIX: localhost]]) by fo2.kc.aoindustries.com (8.13.1/8.13.1/Submit) id jBM1RP7M022699 for dev@forrest.apache.org; Wed, 21 Dec 2005 19:27:25 -0600 X-Authentication-Warning: fo2.kc.aoindustries.com: indexgeo set sender to crossley@apache.org using -f Date: Thu, 22 Dec 2005 12:27:18 +1100 From: David Crossley To: dev@forrest.apache.org Subject: avoiding unnecessary changes in output formatting Message-ID: <20051222012718.GD27762@igg.indexgeo.com.au> References: <20051207195820.40077.qmail@minotaur.apache.org> <43A9582D.6060800@uidesign.de> <499888440512210644l3a8ab75bp8795e0f1fd0df6c6@mail.gmail.com> <43A96B20.90400@uidesign.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <43A96B20.90400@uidesign.de> User-Agent: Mutt/1.4i X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N Johannes Schaefer wrote: > Tim Williams schrieb: > > Johannes Schaefer wrote: > > > >>>Author: rgardler > >>>Date: Wed Dec 7 11:58:13 2005 > >>>New Revision: 354838 > >>> > >>>URL: http://svn.apache.org/viewcvs?rev=354838&view=rev > >>>Log: > >>>check for existence of @class to prevent an extra space being added when not needed > >> > >>Is this necessary? The extra space does not do any harm. > >>If "Yes", I'll update my recent comitts accordingly. > >>Sorry for not noticing/reviewing earlier. > > > > I think this came up because it caused unexpected changes in the > > forrestbot build to our docs. Someone will let us know if I'm wrong. > > http://marc.theaimsgroup.com/?l=forrest-dev&m=113384565318583&w=2 > > wow! small change big effect! > Will do my best to avoid extra spaces, then. > Thanks for the link. Yeah, when i saw our huge commit message for the generated docs i had visions of our users. Many users have the generated docs in version control. So it is important to keep the diffs minimal. Complaints are already heard on other lists about the verbose html output from Forrest. Part of their problem is committers not configuring their svn clients properly, so they see diffs to PDFs too. We have improved a lot lately with our CSS-based design. We had a discussion recently on dev about using a final transformer step at the end to tidy the html output into consistent format. Or perhaps that is the job of the html serializer. Dunno. Anyway should we do that? Users would get one big change after they upgraded tp 0.8 and thereafter the changes would be minimal and output would be consistent. A side-benefit is that it is human-readable. -David