Return-Path: Delivered-To: apmail-xml-cocoon-dev-archive@xml.apache.org Received: (qmail 27252 invoked by uid 500); 8 May 2003 15:42:12 -0000 Mailing-List: contact cocoon-dev-help@xml.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: list-post: Reply-To: cocoon-dev@xml.apache.org Delivered-To: mailing list cocoon-dev@xml.apache.org Received: (qmail 27157 invoked from network); 8 May 2003 15:42:06 -0000 Received: from host190-154.pool80204.interbusiness.it (HELO PC103) (80.204.154.190) by daedalus.apache.org with SMTP; 8 May 2003 15:42:06 -0000 Message-ID: <3EBA79DE.6020206@apache.org> Date: Thu, 08 May 2003 17:38:06 +0200 From: Nicola Ken Barozzi Reply-To: nicolaken@apache.org Organization: Apache Software Foundation User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.4b) Gecko/20030507 X-Accept-Language: en-us, en MIME-Version: 1.0 To: cocoon-dev@xml.apache.org Subject: Re: [Bug] Default-reader depends on JTidy!!! References: In-Reply-To: Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-GCMulti: 1 X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N Carsten Ziegeler wrote, On 08/05/2003 16.26: > Steven Noels wrote: > >>Honestly, I feel non-obvious, hidden 'convenience behaviour' like this >>to be rather awkward. The fact it is located in such an essential part >>of our code without much evidence of it worries me a bit. > > I don't think it's hidden behaviour - it's a contract of the interface: > the interface says "whatever you give me, I'll try to generate SAX events > out of it". > Removing the interface would change the contract to "If you give me > XML I'll generate SAX events out of it". Well, AFAIS the conract (of the Source) should be something like: - if the source is XML, give me SAX - if the source is not XML, give me a Stream What becomes a problem is only this: - if you can make the source in XML, give me SAX This is not a "source"==locator but a generator. -- Nicola Ken Barozzi nicolaken@apache.org - verba volant, scripta manent - (discussions get forgotten, just code remains) ---------------------------------------------------------------------