[Exist-commits] SF.net SVN: exist:[8774] branches/adam/eXist-1.3-hlt/extensions/modules/src /org/e

Revision: 8774
          http://exist.svn.sourceforge.net/exist/?rev=8774&view=rev
Author:   deliriumsky
Date:     2009-03-29 15:41:13 +0000 (Sun, 29 Mar 2009)

Log Message:
-----------
[bugfix] Return XMLReader to pool

Modified Paths:
--------------
    branches/adam/eXist-1.3-hlt/extensions/modules/src/org/exist/xquery/modules/ModuleUtils.java

Modified: branches/adam/eXist-1.3-hlt/extensions/modules/src/org/exist/xquery/modules/ModuleUtils.java
===================================================================
--- branches/adam/eXist-1.3-hlt/extensions/modules/src/org/exist/xquery/modules/ModuleUtils.java	2009-03-29 09:45:04 UTC (rev 8773)
+++ branches/adam/eXist-1.3-hlt/extensions/modules/src/org/exist/xquery/modules/ModuleUtils.java	2009-03-29 15:41:13 UTC (rev 8774)
@@ -83,11 +83,12 @@
 	public static NodeValue streamToXML( XQueryContext context, InputStream xml ) throws XPathException, SAXException 
 	{
 		context.pushDocumentContext();
-		
+
+        XMLReader reader = null;
 		try {
 			// try and construct xml document from input stream, we use eXist's
 			// in-memory DOM implementation
-            XMLReader reader= context.getBroker().getBrokerPool().getParserPool().borrowXMLReader();
+            reader = context.getBroker().getBrokerPool().getParserPool().borrowXMLReader();
             LOG.debug( "Parsing XML response ..." );
 			
 			// TODO : we should be able to cope with context.getBaseURI()
@@ -105,6 +106,9 @@
 		} 
 		finally {
 			context.popDocumentContext();
+            
+            if(reader != null);
+                context.getBroker().getBrokerPool().getParserPool().returnXMLReader(reader);
 		}
 	}
 	


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.