From: Michael Smith <xmldoc@users.sourceforge.net>
Date: Wed, 1 Jun 2005 17:10:03 +0000 (+0000)
Subject: Applied patch from David Green for #1211477, to prevent
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=76ce2927a06c6df327974892d008fd844ba82bd4;p=docbook-dsssl

Applied patch from David Green for #1211477, to prevent
StackOverflowError encountered when processing tables with ~700
rows with Xalan. Smoke-tested and didn't see any obvious problems
with the fix, so going ahead and committing it so others can test
with snapshot.
---

diff --git a/xsl/html/table.xsl b/xsl/html/table.xsl
index 00f650387..51fc48174 100644
--- a/xsl/html/table.xsl
+++ b/xsl/html/table.xsl
@@ -511,7 +511,7 @@
       </xsl:attribute>
     </xsl:if>
 
-    <xsl:apply-templates select="row[1]">
+    <xsl:apply-templates select="row">
       <xsl:with-param name="spans">
         <xsl:call-template name="blank.spans">
           <xsl:with-param name="cols" select="../@cols"/>
@@ -523,7 +523,16 @@
 </xsl:template>
 
 <xsl:template match="row">
-  <xsl:param name="spans"/>
+  <xsl:param name="spans">
+    <!-- Doing it this way in attempt to work around StackOverflowError -->
+    <!-- encountered when processing tables with ~700 rows with Xalan. -->
+    <!-- See #1211477. -->
+    <xsl:if test="preceding-sibling::row">
+      <xsl:apply-templates select="(preceding-sibling::row/entry|preceding-sibling::row/entrytbl)[1]" mode="span">
+        <xsl:with-param name="spans" select="$spans"/>
+      </xsl:apply-templates>
+    </xsl:if>
+  </xsl:param>
 
   <xsl:variable name="row-height">
     <xsl:if test="processing-instruction('dbhtml')">
@@ -613,17 +622,7 @@
     </xsl:apply-templates>
   </tr>
 
-  <xsl:if test="following-sibling::row">
-    <xsl:variable name="nextspans">
-      <xsl:apply-templates select="(entry|entrytbl)[1]" mode="span">
-        <xsl:with-param name="spans" select="$spans"/>
-      </xsl:apply-templates>
-    </xsl:variable>
 
-    <xsl:apply-templates select="following-sibling::row[1]">
-      <xsl:with-param name="spans" select="$nextspans"/>
-    </xsl:apply-templates>
-  </xsl:if>
 </xsl:template>
 
 <xsl:template match="entry|entrytbl" name="entry">