This sounds great. In particular I think having a pluggable EntityResolver on the DefaultNavigator for concrete implementations to use sounds like a great idea.
Do you want to go ahead and make these changes yourself in CVS? If so, mail me your SourceForge user ID and I'll add you as a committer... James ----- Original Message ----- From: "Paul Brown" <[EMAIL PROTECTED]> To: "Jaxen-Interest (E-mail)" <[EMAIL PROTECTED]> Sent: Thursday, May 30, 2002 9:02 PM Subject: [Jaxen] suggestion for enhancement in DOM Navigator Right now, the getDocument() function looks like: public Object getDocument(String uri) throws FunctionCallException { try { DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance(); factory.setNamespaceAware(true); DocumentBuilder builder = factory.newDocumentBuilder(); return builder.parse( uri ); } catch (Exception e) { throw new FunctionCallException("Failed to parse doucment for URI: " + uri, e); } } I would like to suggest the following enhancements: 1) Cache the DocumentBuilderFactory and/or DocumentBuilder once created. (What is the status of a Navigator with respect to threads?) 2) Supply a method to register an EntityResolver instance with the Navigator so that custom resolution is possible. I'll implement them and submit some code if there is agreement on what to do across the different navigators. Thanks in advance. Paul Brown http://www.fivesight.com _______________________________________________________________ Don't miss the 2002 Sprint PCS Application Developer's Conference August 25-28 in Las Vegas -- http://devcon.sprintpcs.com/adp/index.cfm _______________________________________________ Jaxen-interest mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/jaxen-interest _________________________________________________________ Do You Yahoo!? Get your free @yahoo.com address at http://mail.yahoo.com _______________________________________________________________ Don't miss the 2002 Sprint PCS Application Developer's Conference August 25-28 in Las Vegas -- http://devcon.sprintpcs.com/adp/index.cfm _______________________________________________ Jaxen-interest mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/jaxen-interest