Return-Path: Delivered-To: apmail-cocoon-dev-archive@www.apache.org Received: (qmail 93929 invoked from network); 23 Apr 2004 13:27:54 -0000 Received: from daedalus.apache.org (HELO mail.apache.org) (208.185.179.12) by minotaur-2.apache.org with SMTP; 23 Apr 2004 13:27:54 -0000 Received: (qmail 45973 invoked by uid 500); 23 Apr 2004 13:27:45 -0000 Delivered-To: apmail-cocoon-dev-archive@cocoon.apache.org Received: (qmail 45932 invoked by uid 500); 23 Apr 2004 13:27:44 -0000 Mailing-List: contact dev-help@cocoon.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: list-post: Reply-To: dev@cocoon.apache.org Delivered-To: mailing list dev@cocoon.apache.org Received: (qmail 45918 invoked from network); 23 Apr 2004 13:27:44 -0000 Received: from unknown (HELO brown.csi.cam.ac.uk) (131.111.8.14) by daedalus.apache.org with SMTP; 23 Apr 2004 13:27:44 -0000 Received: from wolfbane.caret.cam.ac.uk ([131.111.21.16] helo=caret.cam.ac.uk) by brown.csi.cam.ac.uk with esmtp (Exim 4.20) id 1BH0hp-00078T-Gu for dev@cocoon.apache.org; Fri, 23 Apr 2004 14:27:01 +0100 Received: from agrimony.dmz.caret.local ([192.168.101.161] helo=agrimony.caret.cam.ac.uk) by caret.cam.ac.uk with esmtp (Exim 4.22) id 1BH0dB-0001qX-A9 for dev@cocoon.apache.org; Fri, 23 Apr 2004 14:22:13 +0100 Received: from cantab.net [192.168.101.11] by agrimony.caret.cam.ac.uk with ESMTP (SMTPD32-8.04) id A98F4070056; Fri, 23 Apr 2004 14:26:39 +0100 Message-ID: <408919B2.8090206@cantab.net> Date: Fri, 23 Apr 2004 14:27:14 +0100 From: Andrew Thornton User-Agent: Mozilla Thunderbird 0.5 (X11/20040306) X-Accept-Language: en-us, en MIME-Version: 1.0 To: dev@cocoon.apache.org Subject: Re: XMLDBSource XPathQueries References: <40853DAA.8060805@caret.cam.ac.uk> <40857BDE.7070501@reverycodes.com> <4088DCB0.8080409@cantab.net> <1082719781.14373.170.camel@123.13 yum.ot yum In-Reply-To: <1082719781.14373.170.camel@123.13 yum.ot yum Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-CARET-Spam-Score: -1.0 (-) X-CARET-Spam-Report: -1.0/5.0 ---- Start SpamAssassin results -1.00 points, 5 required; * 0.7 -- From: ends in numbers * -0.5 -- Has a In-Reply-To header * -0.1 -- Has a X-Accept-Language header * -0.5 -- BODY: Contains what looks like an email attribution * -0.5 -- BODY: Contains what looks like a quoted email text * 0.4 -- URI: Includes a link to a likely spammer email address * -0.5 -- Reply with quoted text * 0.0 -- Has a User-Agent header ---- End of SpamAssassin results X-CARET-Scan-Signature: 58ce3bdcd398583bf30ab5ef65931e1c X-Cam-ScannerInfo: http://www.cam.ac.uk/cs/email/scanner/ X-Cam-AntiVirus: No virus found X-Cam-SpamDetails: Not scanned X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N X-Spam-Rating: minotaur-2.apache.org 1.6.2 0/1000/N Bruno Dumon wrote: > Maybe you could let a factory create the PointerParts, and pass the > factory as an argument to the constructor of the parser. Seems like that is probably the best idea. > Or you could indeed move the processing out of these classes, though > that's about the only thing they do... Hmm. XPaths don't process documents there's an XPathProcessor for that. Similarly is it correct the XPointers process Contexts? That being said, I think the XmlnsPart.process() does do the right thing. It sets the namespaces on the Context. Maybe the XPointerPart is trying to do too much. Perhaps the XPointerPart should simply do the XPath with respect to the Context, and not stream? Thanks, andy -- andrew@caret.cam.ac.uk / art27@cantab.net "Absinthe makes the hog Jane Fonda"