You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by mp...@apache.org on 2006/03/22 14:31:11 UTC

svn commit: r387856 - /cocoon/trunk/cocoon-core/src/main/java/org/apache/cocoon/sitemap/DefaultContentAggregator.java

Author: mpfingsthorn
Date: Wed Mar 22 05:31:05 2006
New Revision: 387856

URL: http://svn.apache.org/viewcvs?rev=387856&view=rev
Log:
synch'd change from branch

Modified:
    cocoon/trunk/cocoon-core/src/main/java/org/apache/cocoon/sitemap/DefaultContentAggregator.java

Modified: cocoon/trunk/cocoon-core/src/main/java/org/apache/cocoon/sitemap/DefaultContentAggregator.java
URL: http://svn.apache.org/viewcvs/cocoon/trunk/cocoon-core/src/main/java/org/apache/cocoon/sitemap/DefaultContentAggregator.java?rev=387856&r1=387855&r2=387856&view=diff
==============================================================================
--- cocoon/trunk/cocoon-core/src/main/java/org/apache/cocoon/sitemap/DefaultContentAggregator.java (original)
+++ cocoon/trunk/cocoon-core/src/main/java/org/apache/cocoon/sitemap/DefaultContentAggregator.java Wed Mar 22 05:31:05 2006
@@ -109,29 +109,26 @@
         this.contentHandler.startDocument();
         startElem(this.rootElement);
 
-        try {
-            for (int i = 0; i < this.parts.size(); i++) {
-                final Part part = (Part) this.parts.get(i);
-                this.rootElementIndex = part.stripRootElement ? -1 : 0;
-                if (part.element != null) {
-                    this.currentElement = part.element;
-                    startElem(part.element);
-                } else {
-                    this.currentElement = this.rootElement;
-                }
+        for (int i = 0; i < this.parts.size(); i++) {
+            final Part part = (Part) this.parts.get(i);
+            this.rootElementIndex = part.stripRootElement ? -1 : 0;
+            if (part.element != null) {
+                this.currentElement = part.element;
+                startElem(part.element);
+            } else {
+                this.currentElement = this.rootElement;
+            }
 
-                try {
-                    SourceUtil.parse(this.manager, part.source, this);
-                } finally {
-                    if (part.element != null) {
-                        endElem(part.element);
-                    }
-                }
+            SourceUtil.parse(this.manager, part.source, this);
+            
+            if (part.element != null) {
+                endElem(part.element);
             }
-        } finally {
-            endElem(this.rootElement);
-            this.contentHandler.endDocument();
         }
+
+        endElem(this.rootElement);
+        this.contentHandler.endDocument();
+            
         getLogger().debug("Finished aggregating content");
     }